Open
Conversation
ghostdevv
requested changes
Apr 12, 2026
Collaborator
ghostdevv
left a comment
ghostdevv
left a comment
There was a problem hiding this comment.
Maybe we should link to the other doc, so you can be like "If you only want to parse args, look at this" etc?
Contributor
|
@gameroman i've pushed a change to remove the parseargs replacements as that will be solved by #274 i agree with @ghostdevv too, once the other PR lands and this one is caught up, let's add a link to the parseargs doc. we should also add cleye i think (as a replacement for meow etc) |
ghostdevv
changed the title
ghostdevv
requested changes
Apr 14, 2026
43081j
reviewed
Apr 28, 2026
43081j
reviewed
Apr 28, 2026
Co-authored-by: James Garbutt <43081j@users.noreply.github.com>
43081j
approved these changes
Apr 28, 2026
Contributor
|
@ghostdevv can you re-review, your rejection is blocking merge FYI |
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
3 participants
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.
🔗 Linked issue
Closes #274
📚 Description
Add argument parsers and CLI builders to preferred manifest