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

chore: add release-please and publishing metrics collection #6

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

Conversation

joeldodge79
Copy link
Contributor

No description provided.

Copy link
Contributor Author

@joeldodge79 joeldodge79 left a comment

Choose a reason for hiding this comment

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

even if we don't have the release-please stuff set quite perfectly in this PR, it's easy to change it and get it working correctly once the first "Release PR" is generated (which will happen as soon as there is a user-facing commit

Comment on lines +1 to +13
on:
push:
branches:
- main
name: release-please
jobs:
release-please:
runs-on: ubuntu-latest
steps:
- uses: google-github-actions/release-please-action@v2
with:
command: manifest
token: ${{ secrets.LOS_AUTO_BOT_RP_TOKEN }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

copy/paste from sdk-codegen: enable the release-please action. note: there are a few different ways to configure release-please but I think that as a GH Action using the input command: manifest + a config and .manifest file is the most robust way

we should enable the "semantic pull requests" app/check on this repo to enforce conventional commit formatted commit messages (which release-please relies on for knowing how to bump versions)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

todo: for this and publish-metrics.yml we'll have to add this repo for access to the relevant org GitHub secrets
todo: I think we should only enable the "squash merge" setting (components and sdk-codegen have it, I think it should be our standard)

@@ -0,0 +1,5 @@
{
"packages": {
".": {}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

keys under "packages" are paths that are parent directories to a child package. the "." path is the root path which is the only one in this non-mono-repo. see https://github.com/googleapis/release-please/blob/master/docs/manifest-releaser.md#configfile for a full config file rundown (I don't think we need anything else)

@@ -0,0 +1,5 @@
{
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
{
{
"bootstrap-sha": "fc9873ed5e1baf428b6cb62c8af99b3ad8957c8a",

oops, forgot this. I means release-please will ignore any prior commits. It's a one-time setting that is ignored after the first Release PR is merged and can be safely removed at that point.

@izzymiller
Copy link
Contributor

I think the failure is a flaky test, I need to improve the consistency of them. I'll take a peek a bit later. Thank you for setting this up!

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