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

Add a Maven POM, preconfigured for deploying to https://oss.sonatype.org/ #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

peterjanes
Copy link

This will make it easier to use the plugin with http://jmeter.lazerycode.com/

@jlavallee
Copy link
Owner

pom looks good, but it's missing some of the required bits according to sonatype: http://central.sonatype.org/pages/requirements.html

As I have no plans to publish this project on maven central, perhaps we don't need the distributionManagement section (or indeed, any pom at all)?

@peterjanes
Copy link
Author

I'd encourage you to reconsider publishing to Central. It's not just Maven users who will benefit, but Ant users as well (remember, that's what Ivy uses to retrieve its dependencies).

Depending on your decision, I can fix the POM to update or remove the Sonatype OSS details (it looks like I was using an old reference). Or, if you'd prefer, I can cancel the PR and just maintain my own fork that includes the POM.

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.

2 participants