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

Issue 1818: Improve our checksum support for Maven metadata #1974

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

sainalshah
Copy link
Member

Pull Request Description

This pull request closes #1818

Acceptance Test

  • Building the code with mvn clean install -Dintegration.tests still works.
  • Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
  • Building the code and running the strongbox-distribution from a zip or tar.gz still works.
  • The tests in the strongbox-web-integration-tests still run properly.

Questions

Code Review And Pre-Merge Checklist

  • My code follows the coding convention of this project.
  • I have performed a self-review of my own code.
  • I have commented my code in hard-to-understand areas.
  • My changes generate no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.

@carlspring
Copy link
Member

@sainalshah ,

Thanks for you help! Could you please follow our coding convention and go over your code and apply the necessary changes? I've flagged a few, but there are more and I'm sure you'll spot them once you've had a look at our coding convention and installed the respective code style files.

I'll have another look, when you've applied these. Thanks! :)

Co-authored-by: Martin Todorov <carlspring@gmail.com>
@sainalshah
Copy link
Member Author

@carlspring As we discussed on chat, I have removed the code for global repository configuration both from strongbox and strongbox-web-ui projects.

The compilation on Jenkins is failing because it cannot find the updated carlspring/commons-io artifiact. I'm not sure about the practice here, would you be making a new release for carlspring/commons-io with the latest changes?

@carlspring
Copy link
Member

@sainalshah ,

You need to create a new pull request against the strongbox-parent and change the version of the commons-io there. Then you need to update the current pull request to use that version of the parent. If it works,

  • @steve-todorov : Could you please cut a release of carlspring/commons-io and deploy it to Maven Central
  • @sainalshah :
    • Close the pull request for the strongbox-parent.
    • Revert the current pull request to strongbox-parent:1.0.

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.

Improve our checksum support for Maven metadata
2 participants