-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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 it test for Maven Plugin #1638
Comments
@WalkerWatch please assign to me |
@janbartel I have a branch with the it tests here https://github.com/olamy/jetty.project/tree/feature/maven_plugin_its |
regarding fragments I can see a TODO :-) here https://github.com/eclipse/jetty.project/blob/jetty-9.4.x/jetty-webapp/src/main/java/org/eclipse/jetty/webapp/MetaInfConfiguration.java#L279 |
There's something up with the testcases in master. Probably a bad merge. :-( |
ok I will have a look on master branch |
well it looks master has a lot changes which cause a lot of failure in the maven plugin. |
Signed-off-by: olivier lamy <olamy@webtide.com>
Signed-off-by: olivier lamy <olamy@webtide.com>
Signed-off-by: olivier lamy <olamy@webtide.com>
Signed-off-by: olivier lamy <olamy@webtide.com>
the goal is to have it test for each mojo
The text was updated successfully, but these errors were encountered: