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 tests for #803 #806

Merged
merged 1 commit into from
Sep 8, 2019
Merged

Add tests for #803 #806

merged 1 commit into from
Sep 8, 2019

Conversation

byteit101
Copy link

As requested, some tests for #803. Can't get them to work locally for some reason though, as I commented in #803

@remkop remkop added type: bug 🐛 theme: codegen An issue or change related to the picocli-codegen module labels Sep 8, 2019
@remkop remkop added this to the 4.0.4 milestone Sep 8, 2019
@remkop remkop merged commit 2aed1ee into remkop:master Sep 8, 2019
@remkop
Copy link
Owner

remkop commented Sep 8, 2019

Thank you! Taking a look now...

@remkop
Copy link
Owner

remkop commented Sep 8, 2019

The test helped me quickly zoom in to the cause of the problem (thanks again): the logic to pick up this attribute wasn’t implemented yet in the annotation processor. Working on this now.

remkop added a commit that referenced this pull request Sep 8, 2019
…ustom `IParameterConsumer` caused native-image to fail. Added tests.
@remkop
Copy link
Owner

remkop commented Sep 8, 2019

I pushed a fix for #803 to master.
Can you verify?

@byteit101
Copy link
Author

I have verified that everything works. Thanks!

@remkop
Copy link
Owner

remkop commented Sep 9, 2019

Great!
If you like the project please star it on GitHub and tell your friends! ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme: codegen An issue or change related to the picocli-codegen module type: bug 🐛
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants