Skip to content

[MEAR-216] Handling test JARs as regular JARs#22

Merged
hboutemy merged 13 commits intoapache:masterfrom
mabrarov:MEAR-216
Dec 20, 2020
Merged

[MEAR-216] Handling test JARs as regular JARs#22
hboutemy merged 13 commits intoapache:masterfrom
mabrarov:MEAR-216

Conversation

@mabrarov
Copy link
Copy Markdown
Contributor

@mabrarov mabrarov commented Oct 13, 2020

  • [MEAR-216] - Handling test JARs as regular JARs
  • [MEAR-216] - Support of jboss-sar, jboss-har and jboss-par types of artifacts.

@mabrarov mabrarov changed the title [MEAR-216] - Handling test JARs as regular JARs [MEAR-216] Handling test JARs as regular JARs Oct 13, 2020
@mabrarov
Copy link
Copy Markdown
Contributor Author

@elharo, could you please trigger Jenkins build for this pull request?

@elharo
Copy link
Copy Markdown
Contributor

elharo commented Oct 15, 2020

@mabrarov
Copy link
Copy Markdown
Contributor Author

mabrarov commented Oct 16, 2020

@elharo,

Sorry for being noisy and disrupting you, but it looks like Jenkins build passed successfully. What needs to be done to merge this pull request and close MEAR-216?

@mabrarov
Copy link
Copy Markdown
Contributor Author

Need to check one more case I found recently. The case (I'll add a test for it) may demonstrate that this pull request is wrong and needs further work. Please, don't merge this pull request until I reply with result of my check.

@mabrarov mabrarov marked this pull request as draft October 22, 2020 13:14
@mabrarov
Copy link
Copy Markdown
Contributor Author

I fixed that issue and this pull request is ready for review now.

@elharo and @hboutemy, your help with Jenkins build, review and merging is appreciated.

Thank you.

@mabrarov mabrarov marked this pull request as ready for review October 22, 2020 21:16
# Conflicts:
#	pom.xml
#	src/main/java/org/apache/maven/plugins/ear/HarModule.java
#	src/main/java/org/apache/maven/plugins/ear/RarModule.java
#	src/main/java/org/apache/maven/plugins/ear/SarModule.java
#	src/main/java/org/apache/maven/plugins/ear/WebModule.java
#	src/test/java/org/apache/maven/plugins/ear/it/EarMojoIT.java
@mabrarov
Copy link
Copy Markdown
Contributor Author

mabrarov commented Dec 6, 2020

Merged conflicts were resolved. Please trigger a new Jenkins build.

Copy link
Copy Markdown
Contributor

@elharo elharo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mabrarov
Copy link
Copy Markdown
Contributor Author

mabrarov commented Dec 6, 2020

This PR was successfully built by Jenkins. Is it possible to get this PR merged?

@mabrarov
Copy link
Copy Markdown
Contributor Author

@elharo and @hboutemy, is it possible to get this PR merged? What's missing / pending or blocking this PR?

@hboutemy hboutemy merged commit dbcdb6b into apache:master Dec 20, 2020
@hboutemy
Copy link
Copy Markdown
Member

you're right, merged
thank you

@jira-importer
Copy link
Copy Markdown

Resolve #392

1 similar comment
@jira-importer
Copy link
Copy Markdown

Resolve #392

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.

4 participants