Add infrastructure to sync 16 upstream issues from paulnasca/paulstretch_cpp#1
Conversation
Co-authored-by: peternicholls <519525+peternicholls@users.noreply.github.com>
Co-authored-by: peternicholls <519525+peternicholls@users.noreply.github.com>
Co-authored-by: peternicholls <519525+peternicholls@users.noreply.github.com>
Co-authored-by: peternicholls <519525+peternicholls@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR provides infrastructure to sync 16 issues from the upstream paulnasca/paulstretch_cpp repository into the peternicholls/paulstretch_cpp fork. The implementation includes automation scripts, comprehensive documentation, and complete issue content preservation with proper attribution.
Key Changes
- Adds two automated scripts (Bash using GitHub CLI and Python using PyGithub API) to create 16 upstream issues
- Provides complete documentation in markdown files with usage instructions, troubleshooting guides, and full issue content
- Updates
.gitignoreto exclude Python cache and IDE files
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
scripts/sync_upstream_issues.sh |
Bash script using heredocs to create 16 GitHub issues via gh CLI with proper escaping of special characters |
scripts/create_upstream_issues.py |
Python script using PyGithub library as alternative implementation for issue creation |
scripts/README.md |
Comprehensive usage guide covering both scripts, prerequisites, troubleshooting, and token setup |
SYNC_UPSTREAM_ISSUES.md |
Complete formatted content of all 16 upstream issues for manual reference or creation |
CREATE_ISSUES_INSTRUCTIONS.md |
Quick-start guide with three execution options (GitHub CLI, Python API, or manual) |
.gitignore |
Standard Python, IDE, and OS-specific exclusions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| **Original Issue:** https://github.com/paulnasca/paulstretch_cpp/pull/8 | ||
| **Original Author:** @Chaircrusher | ||
|
|
||
| This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against. |
There was a problem hiding this comment.
Grammatical error in the issue body. The phrase "This should all (still) building" is missing the word "be" and should read "This should all (still) be building on Mac OS X".
| This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against. | |
| This simply adds the CMake build system. This should all (still) be building on Mac OS X, but I no longer have a system to test this build environment against. |
| "title": "C make build system", | ||
| "upstream_link": "https://github.com/paulnasca/paulstretch_cpp/pull/8", | ||
| "author": "Chaircrusher", | ||
| "body": "This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against." |
There was a problem hiding this comment.
Grammatical error in the issue body. The phrase "This should all (still) building" is missing the word "be" and should read "This should all (still) be building on Mac OS X".
| "body": "This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against." | |
| "body": "This simply adds the CMake build system. This should all (still) be building on Mac OS X, but I no longer have a system to test this build environment against." |
| **Original Issue:** https://github.com/paulnasca/paulstretch_cpp/pull/8 | ||
| **Original Author:** @Chaircrusher | ||
|
|
||
| This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against. |
There was a problem hiding this comment.
Grammatical error in the issue body. The phrase "This should all (still) building" is missing the word "be" and should read "This should all (still) be building on Mac OS X".
| This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against. | |
| This simply adds the CMake build system. This should all (still) be building on Mac OS X, but I no longer have a system to test this build environment against. |
|
|
||
| ## Notes | ||
|
|
||
| - These scripts are idempotent-safe, meaning running them multiple times will create duplicate issues. Only run once unless you want duplicates. |
There was a problem hiding this comment.
The statement "These scripts are idempotent-safe, meaning running them multiple times will create duplicate issues" is contradictory and confusing. Idempotent operations can be run multiple times without creating duplicates. The correct phrasing should be: "These scripts are NOT idempotent, meaning running them multiple times will create duplicate issues. Only run once unless you want duplicates."
| - These scripts are idempotent-safe, meaning running them multiple times will create duplicate issues. Only run once unless you want duplicates. | |
| - These scripts are NOT idempotent, meaning running them multiple times will create duplicate issues. Only run once unless you want duplicates. |
Provides scripts and documentation to create 16 issues from upstream
paulnasca/paulstretch_cpprepository with proper attribution and formatting.Files Added
SYNC_UPSTREAM_ISSUES.md- Full issue content for all 16 upstream issuesscripts/sync_upstream_issues.sh- Bash script usingghCLI (preferred)scripts/create_upstream_issues.py- Python script using PyGithub APIscripts/README.md- Usage instructions and troubleshootingCREATE_ISSUES_INSTRUCTIONS.md- Quick-start guide.gitignore- Excludes Python cacheIssue Format
Each issue includes:
@dblanchemain)[Synced from upstream]prefixExecution
Scripts use heredoc/file-based approach to safely handle special characters in issue bodies. Both syntax-validated.
Issues Covered
16 total: segfault (paulnasca#18), GCC14 undefined behavior (PR#17), Mac signing (paulnasca#15), mxml 3.0 compat (PR#12), CMake build (PR#8), FLAC support (PR#7), and 10 others spanning documentation, build fixes, and feature requests.
Original prompt
This pull request was created as a result of the following prompt from Copilot chat.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.