Repository navigation
fix(scripts): check prerequisites before issue mutations - #406
Open
woahwhattheheck wants to merge 1 commit into
Open
woahwhattheheck wants to merge 1 commit into
woahwhattheheck wants to merge 1 commit into
Conversation
Contributor
|
@woahwhattheheck is attempting to deploy a commit to the christabel888's projects Team on Vercel. A member of the Team first needs to authorize it. |
Author
|
I am claiming consideration for any GrantFox reward applicable to my contribution in this PR for #323, payable to @woahwhattheheck. Please confirm that this PR is associated with the original contributor's campaign record and included in the reward evaluation, and confirm any allocation and payout date once approved. Existing contributors' attribution and any remaining acceptance requirements remain documented in this PR. |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The legacy issue generator assumes backend, SDK and mobile source roots in one checkout, while this frontend tree lacks
backend/srcandmobile/src. The recreation wrapper also closes issues before checking for its named replacement script,create_all_70_detailed.py, which is absent from this tree.This focused change checks the generator's source roots before its first issue-creation call, anchors its existing subprocess to the script's checkout, and stops the recreation wrapper before any
ghcall when its named replacement is missing or unreadable. The task payloads, closure loop and manual recreation command remain unchanged. This does not implement backend issue #323 or make issue closure/recreation transactional.Validation: 3 offline regression tests pass (
python -m unittest discover -s scripts/tests -v), including a wrapper invocation from an unrelated directory with a localghsentinel proving noghinvocation occurs when the replacement is absent.bash -nand Python compilation pass. No real GitHub CLI issue listing, creation or closure was executed.The scripts' executable modes are preserved; original repository authorship is retained.