-
Notifications
You must be signed in to change notification settings - Fork 28
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 java release commands #29
Conversation
Love this. Gonna try to review today after I solve some bugs here. |
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.
Looking mostly ready, just some small things.
releasetool/commands/start/java.py
Outdated
|
||
def gather_changes(ctx: Context) -> None: | ||
click.secho(f"> Gathering changes since {ctx.last_release_version}", fg="cyan") | ||
ctx.changes = releasetool.git.summary_log(ctx.last_release_committish) |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
releasetool/commands/start/java.py
Outdated
|
||
def gather_pom_xml_files(ctx: Context) -> None: | ||
ctx.pom_files = glob('**/pom.xml', recursive=True) | ||
print(ctx.pom_files) |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
releasetool/commands/start/java.py
Outdated
|
||
def update_pom_xml(ctx: Context) -> None: | ||
click.secho("> Updating snapshot versions in pom.xml files.", fg="cyan") | ||
print(ctx.snapshot_version) |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
releasetool/commands/tag/java.py
Outdated
if match is not None: | ||
ctx.release_tag = match.group(1) | ||
else: | ||
print( |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
b1fa276
to
c88c41d
Compare
Thank you! |
The general process for java is:
Java libraries do not maintain a running CHANGELOG.md (can revisit this later) so we put the release notes in the PR body and extract them for the GitHub release.
Example PR: googleapis/google-auth-library-java#179