-
Notifications
You must be signed in to change notification settings - Fork 11
feat: add parsed meta-data to returned properties #129
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
69 commits
Select commit
Hold shift + click to select a range
52c3a2e
Proof of concept
shadowspawn 5ec71b1
Return originalArgs and indices
shadowspawn f711187
Change short in AST to boolean
shadowspawn cce90bb
Store inlineValue rather than valueIndex
shadowspawn e4bc5fe
Rename symbol property in AST to kind
shadowspawn 87624a1
Rename returned ast property to parseElements
shadowspawn 2925639
Refactor to use parseElements to test for errors and store values
shadowspawn 412287e
Merge remote-tracking branch 'upstream/main' into feature/ast
shadowspawn 1897dac
Build positionals from parseElements
shadowspawn 45d12e7
Replace .push with primordial
shadowspawn 1d19fb2
forEach replaced with primordial
shadowspawn 9d1267d
Swap isShort for optionUsed
shadowspawn c9c4d4c
Rework checkOptionLikeValue
shadowspawn ed4168d
Consistent property order and renames
shadowspawn 16a9f51
Comment out new returned property until naming and tests ready
shadowspawn 915e6c3
Refactor tokenize into own function
shadowspawn 641197e
First test pass, get tests working by ignoring tokens in results
shadowspawn 18470bd
Less magical check now checking kind
shadowspawn 38d1d6c
Add some token tests
shadowspawn cfbd436
Fix index after space-separated option value
shadowspawn 397dcf1
Make tokens opt-in
shadowspawn 938bdb0
Simplify more tests with opt-in details
shadowspawn c5da345
Rename token.optionName to name, and restore longOption/shortOption …
shadowspawn 4e1ec2d
Add example
shadowspawn bd3f574
Add side-note
shadowspawn b703ee8
Add example of #52, limit long syntax
shadowspawn 82339f9
Add ordered example
shadowspawn fab1dc4
Add example for no repeated options
shadowspawn 99165c9
Comment wording
shadowspawn 9bd039d
Switch from details to tokens for configuration
shadowspawn 6a3f637
Simplify example by removing "library" support
shadowspawn 4cfbc29
Remove comments on how well-advised the examples goals are
shadowspawn 9a9a740
Switch from optionUser to rawName
shadowspawn 7cd685c
Merge remote-tracking branch 'upstream/main' into feature/ast
shadowspawn 6f93632
Use modern syntax
shadowspawn 642d8c0
Validate new input property
shadowspawn e70609a
Move strict check outside check routines
shadowspawn fdaa553
Tidy object setup
shadowspawn c293307
Simplify test
shadowspawn 041459d
Indentation and match filename
shadowspawn 7aafaf6
Rework with strict:true
shadowspawn 1b6e585
Longer but simpler
shadowspawn 20eacb0
Add textual description, and some fixes
shadowspawn 81239d6
Add example output for tokens
shadowspawn 84dde8e
Add example used in new documentation
shadowspawn 40a6201
Refactor documentation
shadowspawn 07fbb91
Update .editorconfig from Node.js
shadowspawn 35c16f1
rework token property descriptions
shadowspawn 038480a
Minor refactor of remaining arg processing
shadowspawn d4f96cc
Replace switch with if/else
shadowspawn 122727e
Remove side-affect, per feedback
shadowspawn 5b3518d
Add tricky case of short option group to token expansion
shadowspawn c8c2f84
Rework description of token.index after token example.
shadowspawn 2b119b0
Be less clever with script naming in example
shadowspawn 1b3c076
Remove superfluous colons in docs
shadowspawn e9e30a7
Upstream lint
shadowspawn 46ab295
Merge branch 'main' into feature/ast
shadowspawn 01baee5
Merge branch 'main' into feature/ast
shadowspawn 46903af
Improve description
shadowspawn 9b74c05
Merge branch 'feature/ast' of github.com:shadowspawn/parseargs into f…
shadowspawn 89e4532
Use negate as example for tokens. Rework description a little.
shadowspawn 3914949
Lint and feedback
shadowspawn d8126d1
Expand small token tests and remove uber tests
shadowspawn 87bab7b
Use kEmptyObject per upstream suggestion. Move node lookalikes to int…
shadowspawn 02ea585
Reworks tokens documentation for config
shadowspawn 56fe4ca
Add version changes to YAML.
shadowspawn 722f05e
Update README with upstream changes
shadowspawn ae9eca4
Update negate example calls to match documentation
shadowspawn f9dcc4f
Remove extra examples
shadowspawn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
forEach replaced with primordial
- Loading branch information
commit 1d19fb21dc471c269aeafd1b62c8f8d6df06f74a
There are no files selected for viewing
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
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.
Uh oh!
There was an error while loading. Please reload this page.