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

[UNDERTOW-2416] Make the script fail if there are compilation errors … #1615

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

fl4via
Copy link
Member

@fl4via fl4via commented Jun 23, 2024

@fl4via fl4via added the enhancement Enhances existing behaviour or code label Jun 23, 2024
…in tests

Signed-off-by: Flavia Rainone <frainone@redhat.com>
@fl4via
Copy link
Member Author

fl4via commented Jun 24, 2024

I'll repen after my PTO

@fl4via fl4via closed this Jun 24, 2024
@fl4via
Copy link
Member Author

fl4via commented Jul 14, 2024

Restarting to work on this PR so I can fix it...

@fl4via fl4via reopened this Jul 14, 2024
@fl4via fl4via added the under verification Currently being verified (running tests, reviewing) before posting a review to contributor label Jul 14, 2024
@ropalka
Copy link
Contributor

ropalka commented Aug 6, 2024

LGTM . It catches also javadoc errors but that is OK IMHO @fl4via

@ropalka ropalka requested a review from baranowb August 7, 2024 12:58
@baranowb
Copy link
Contributor

baranowb commented Aug 22, 2024

This will make build fail, most likely due to https://issues.apache.org/jira/browse/MJAVADOC-623

INdeed, bumping plugin to 3.3.0 fix this problem.
Related:
https://issues.redhat.com/browse/UNDERTOW-2431 ,https://issues.redhat.com/browse/UNDERTOW-2432

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhances existing behaviour or code under verification Currently being verified (running tests, reviewing) before posting a review to contributor
Projects
None yet
3 participants