Skip to content
This repository has been archived by the owner on Feb 23, 2022. It is now read-only.

maven central requirements #6

Merged
merged 2 commits into from
Apr 11, 2017

Conversation

alexanderkjall
Copy link
Contributor

I went over these guides http://central.sonatype.org/pages/requirements.html and http://central.sonatype.org/pages/apache-maven.html and added what was required getting the project accepted into maven central.

Someone also needs to apply for an account at OSSRH as described here: http://central.sonatype.org/pages/ossrh-guide.html .

I also added a -SNAPSHOT to the version, this is a magic word for maven that tells it that it's not a release that is being built. My thought was that when we want to release a new version, we branch the repo, remove the -SNAPSHOT and upload that.

The javadoc step adds some warnings about javadoc problems, I think those can be cleaned up in a separate commit?

@codingllama
Copy link
Contributor

Hey Alexander, sorry for the delay. I won't be able to have a look today, but I'll try to do at least a first pass early next week.

Copy link
Contributor

@codingllama codingllama left a comment

Choose a reason for hiding this comment

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

A few nits, but LGTM.

Re Javadoc warnings: yes, let's address those separately. (#7)

Re OSSRH account: would you mind raising an issue asking for it? Meanwhile I'll double check it with the team; people may want a few changes before it.

Many thanks for the contribution, Alexander!

pom.xml Outdated

<name>${project.groupId}:${project.artifactId}</name>
<description>A application used to communicate with certificate transparency log servers.</description>
<url>http://certificate-transparency.org/</url>
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: https

pom.xml Outdated
<licenses>
<license>
<name>The Apache License, Version 2.0</name>
<url>http://www.apache.org/licenses/LICENSE-2.0.txt</url>
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: https

@alexanderkjall
Copy link
Contributor Author

I can submit a request to OSSRH, it will be the first time I do this myself, so it will be a good learning experience.

@codingllama codingllama merged commit 8e77d82 into google:master Apr 11, 2017
@codingllama
Copy link
Contributor

Thanks for the changes, Alexander.

About OSSRH, I think one of the open source owners should do it. I'd also like to run it by our tech lead before signing in, have a look at the T&C, etc. Unfortunately a few people are on vacation this week, including our TL, so I'll ask you to wait for about a week on it.

Raised #9.

@alexanderkjall
Copy link
Contributor Author

I agree that it's better if one of the owners do it. I would recommend some sort of system so that the same gpg key is used to sign all builds going forward. That makes it easier for people to verify the integrity of the builds.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants