Skip to content

ci: adding a non-required check to ensure no existing version on release PR - #8788

Merged
suztomo merged 14 commits into
mainfrom
check_existing_release_versions
Nov 15, 2022
Merged

ci: adding a non-required check to ensure no existing version on release PR#8788
suztomo merged 14 commits into
mainfrom
check_existing_release_versions

Conversation

@suztomo

@suztomosuztomo commented Nov 15, 2022

Copy link
Copy Markdown
Member

This check runs on Release Please pull request and ensures the followings:

  • There's no "SNAPSHOT" version in our artifacts .
  • The artifacts versions do not exist in Maven Central.

Example invocation using the branch of the Release Please pull request (#8686): https://gist.github.com/suztomo/4f2c5d03d5b4007ca6f6e1b17af7367d

@suztomo
suztomo marked this pull request as draft November 15, 2022 03:18
@suztomo

Copy link
Copy Markdown
MemberAuthor

Still working on it.

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>com.google.cloud</groupId>

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

For the sake of consistency, adding this groupId element. "com.google.cloud" is the groupID of this artifact:
https://search.maven.org/artifact/com.google.cloud/google-cloud-notification

@suztomo
suztomo marked this pull request as ready for review November 15, 2022 04:51
@suztomosuztomo changed the title ci: adding a check to ensure no existing versionci: adding a non-required check to ensure no existing version on release PRNov 15, 2022
@suztomosuztomo added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 15, 2022
@gcf-owl-botgcf-owl-botBot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 15, 2022
@suztomo
suztomo merged commit 4cd1bab into mainNov 15, 2022
@suztomo
suztomo deleted the check_existing_release_versions branch November 15, 2022 14:54
Sign up for freeto 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

@suztomo@burkedavison