-
Notifications
You must be signed in to change notification settings - Fork 6
Update pom for release #37
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
Conversation
@@ -3,7 +3,7 @@ | |||
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" | |||
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"> | |||
<modelVersion>4.0.0</modelVersion> | |||
<groupId>dev.cdevents.sdk-java</groupId> | |||
<groupId>dev.cdevents</groupId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
\o/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
@aalmiray - sorry, it looks like this may need a rebase now. |
It's OK. I'll take care of it in a few. |
<version.gpg.plugin>1.6</version.gpg.plugin> | ||
<version.install.plugin>3.0.0-M1</version.install.plugin> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A personal question, did maven release a specific version for the Mac M1 chip? Have not seen this version and tried looking it up but could not find an answer for the M1 on the plugin version. Very interesting!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not a chip related version. M1 usually stands for "milestone". It's just a version tag like GA, FINAL, and so forth.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's fantastic, thanks for sharing! I just haven't seen an M1 version of a plugin before and was excited to learn about it. 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow I learned a lot in this pull request and had to research a few of the plugin changes! Thanks for making such an awesome contribution! Looks great to me!
🎉 This issue has been resolved in |
Fixes #32
Changes made in small increments for better review.
Please squash all changes before merging.