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

Moved hardcoded dependency versions to properties tag in pom.xml #1583

Merged
merged 1 commit into from
Aug 24, 2018

Conversation

FatihErdem
Copy link
Contributor

Please answer these questions before submitting pull request

  • Why submit this pull request?
  • Bug fix
  • New feature provided
  • Improve performance

New feature or improvement

  • Describe the details and related test reports.

In all pom's dependency versions were hardcoded. I moved them into properties tag. I scan the whole project.

@wu-sheng
Copy link
Member

@FatihErdem You are really persistent to mive the version to pom properties.😄

@FatihErdem
Copy link
Contributor Author

@wu-sheng Haha. Yes. Maybe someday I can make a real contribution to the project. 😄

@wu-sheng
Copy link
Member

Haha. Yes. Maybe someday I can make a real contribution to the project.

This is a contribution too.

@wu-sheng wu-sheng added this to the 5.0.0-RC milestone Aug 24, 2018
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.004%) to 24.719% when pulling c9b23ea on FatihErdem:pom-refactor-2 into 999762e on apache:master.

@wu-sheng wu-sheng merged commit 15263bc into apache:master Aug 24, 2018
@FatihErdem FatihErdem deleted the pom-refactor-2 branch October 31, 2018 15:54
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