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

add test for #964 #1080

Merged
merged 1 commit into from Feb 8, 2022
Merged

add test for #964 #1080

merged 1 commit into from Feb 8, 2022

Conversation

Lyther
Copy link
Contributor

@Lyther Lyther commented May 30, 2020

Add a simple test for issue #964 in synopsis order. Test in simple @Option is behave normally but order doesn't correct if combine @ArgGroup and @Option.

@remkop
Copy link
Owner

remkop commented May 31, 2020

@Lyther, thank you for the pull request!
I have some other work in progress, it may take me some time before I can start looking at #964 and this PR. So please be don't be discouraged if you don't hear anything for a while... 😃

@Lyther
Copy link
Contributor Author

Lyther commented May 31, 2020

Thanks for replying! I'm working on issue 964 currently. Maybe next time you can find a pr that solved that issue.

@remkop remkop merged commit c008dd3 into remkop:master Feb 8, 2022
@remkop remkop added this to the 4.6.3 milestone Feb 8, 2022
@remkop
Copy link
Owner

remkop commented Feb 8, 2022

Sorry for the very, very long wait. 😅
I merged the PR.
Thank you very much for the contribution! 🙏

@remkop remkop mentioned this pull request Feb 8, 2022
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.

2 participants