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

[MNG-6759] - Add test demonstrating the issue where the wrong reposit… #50

Merged
merged 3 commits into from Nov 17, 2019

Conversation

srdo
Copy link
Contributor

@srdo srdo commented Oct 12, 2019

…ories are used for resolving transitive dependencies

Test for apache/maven#294.

Advice on improving these would be appreciated. It's not good to be depending on an external plugin or an external repository.

@srdo srdo force-pushed the MNG-6759 branch 2 times, most recently from 10757af to b3726ca Compare October 18, 2019 23:51
…ories are used for resolving transitive dependencies
@srdo
Copy link
Contributor Author

srdo commented Oct 18, 2019

Fixed so the test now has no external dependencies.

@rfscholte
Copy link
Contributor

Don't forget to add your test to org.apache.maven.it.IntegrationTestSuite

@srdo
Copy link
Contributor Author

srdo commented Oct 22, 2019

Thanks for letting me know about the suite. Addressed your comments.

@rfscholte rfscholte merged commit 9e98d89 into apache:master Nov 17, 2019
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