Skip to content

Update maven-dependency-plugin to version 3.0.1#3219

Merged
asfgit merged 1 commit into
apache:masterfrom
iemejia:update-dependency-plugin
May 25, 2017
Merged

Update maven-dependency-plugin to version 3.0.1#3219
asfgit merged 1 commit into
apache:masterfrom
iemejia:update-dependency-plugin

Conversation

@iemejia
Copy link
Copy Markdown
Member

@iemejia iemejia commented May 24, 2017

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

  • Make sure the PR title is formatted like:
    [BEAM-<Jira issue #>] Description of pull request
  • Make sure tests pass via mvn clean verify.
  • Replace <Jira issue #> in the title with the actual Jira issue
    number, if there is one.
  • If this contribution is large, please file an Apache
    Individual Contributor License Agreement.

@iemejia
Copy link
Copy Markdown
Member Author

iemejia commented May 24, 2017

R: @jbonofre
This is a trivial one just updated the plugin because it addresses
https://issues.apache.org/jira/browse/MDEP-409
And some other fixes.

Copy link
Copy Markdown
Member

@jbonofre jbonofre left a comment

Choose a reason for hiding this comment

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

LGTM. I'm waiting Jenkins pass before merging.

@iemejia iemejia force-pushed the update-dependency-plugin branch from da11a5d to 8b0e4bc Compare May 25, 2017 04:38
@iemejia
Copy link
Copy Markdown
Member Author

iemejia commented May 25, 2017

I had to rebase to add two extra missing dependencies found by the new version of the plugin.
It should be ok now.

@jbonofre
Copy link
Copy Markdown
Member

retest this please

@jbonofre
Copy link
Copy Markdown
Member

The build fails on python, I'm checking on my machine.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.004%) to 70.705% when pulling 8b0e4bc on iemejia:update-dependency-plugin into c0d19f9 on apache:master.

@iemejia
Copy link
Copy Markdown
Member Author

iemejia commented May 25, 2017

It looks good now it seems.

@jbonofre
Copy link
Copy Markdown
Member

It looks good now !

@iemejia
Copy link
Copy Markdown
Member Author

iemejia commented May 25, 2017

Can you merge it, or do I?

@asfgit asfgit merged commit 8b0e4bc into apache:master May 25, 2017
asfgit pushed a commit that referenced this pull request May 25, 2017
@jbonofre
Copy link
Copy Markdown
Member

Done

@iemejia
Copy link
Copy Markdown
Member Author

iemejia commented May 25, 2017

Thx JB.

@iemejia iemejia deleted the update-dependency-plugin branch May 26, 2017 13:34
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