Skip to content
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

traverse: fix shorthand chain #840

Merged
merged 1 commit into from
Jul 25, 2023
Merged

Conversation

rsteube
Copy link
Member

@rsteube rsteube commented Jul 24, 2023

fix #733
fix #504

@rsteube rsteube force-pushed the traverse-fix-shorthandchain branch 3 times, most recently from c10e08e to ab59a58 Compare July 24, 2023 23:28
@rsteube rsteube changed the title traverse fix shorthandchain traverse: fix shorthandchain Jul 25, 2023
@rsteube rsteube force-pushed the traverse-fix-shorthandchain branch from fb47b5b to 19c6014 Compare July 25, 2023 08:33
@rsteube rsteube changed the title traverse: fix shorthandchain traverse: fix shorthand chain Jul 25, 2023
- support "attached" value flags like `-v=arg` and `-varg`
- use shorthand for nospace in shorthand series

Some logs are wrong and there are likely some edge cases failing.
But tests are fine and works well so far so issues will be fixed in subsequent PRs.
@rsteube rsteube force-pushed the traverse-fix-shorthandchain branch from 707d6b1 to d0887d3 Compare July 25, 2023 10:57
@rsteube rsteube marked this pull request as ready for review July 25, 2023 10:57
@rsteube rsteube merged commit 2b0183d into master Jul 25, 2023
8 checks passed
@rsteube rsteube deleted the traverse-fix-shorthandchain branch July 25, 2023 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

traverse: shorthand argument without space traverse: shorthand optarg
1 participant