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

Update readme to make it more clear that you need to apply plugin as well in legacy syntax #314

Closed
wants to merge 1 commit into from

Conversation

peterdk
Copy link
Contributor

@peterdk peterdk commented Feb 4, 2024

I was struggling for more then an hour to get my Junit5 tests to recognize. Turns out that I didn't see the subtle suggestion in the readme that you also need to apply the plugin. Closes #313

So this PR makes it more explicit.

@peterdk peterdk changed the title update readme to make it more clear that you need to apply plugin as well in legacy syntax Update readme to make it more clear that you need to apply plugin as well in legacy syntax Feb 4, 2024
@mannodermaus
Copy link
Owner

Thank you! I've taken the liberty to make a tweak to the proposed change and will merge it as #315, closing this one. 🙇‍♂️

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.

Readme should include apply plugin for legacy syntax
2 participants