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

[MASSEMBLY-969] Update common artifact filters #83

Merged
merged 4 commits into from Jul 20, 2022
Merged

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Jul 16, 2022

As version 3.3.0 has an unwanted change regarding 4 elements
pattern and are parsed as GATV only, while before they were
parsed as GATC as well.

Created IT based on issue supplied POM and assembly descriptor.


https://issues.apache.org/jira/browse/MASSEMBLY-969
Related PR apache/maven-common-artifact-filters#29

As version 3.3.0 has an unwanted change regarding 4 elements
pattern and are parsed as GATV only, while before they were
parsed as GATC as well.
@cstamas cstamas self-assigned this Jul 16, 2022
@cstamas
Copy link
Member Author

cstamas commented Jul 16, 2022

The IT failed as expected, as apache/maven-common-artifact-filters#29 is not yet merged, hence, fix is not yet deployed

This should make new IT pass as well
@cstamas
Copy link
Member Author

cstamas commented Jul 20, 2022

Updated to released common filters 3.3.1 and now the new IT (added by this PR) should pass, again, as expected.

@cstamas cstamas merged commit db292ea into master Jul 20, 2022
@cstamas cstamas deleted the MASSEMBLY-969 branch July 20, 2022 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants