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

fix(ci): CHANGELOG.md generation fix for 'no change' and yoshi-code-bot email #9566

Merged
merged 3 commits into from
Jun 21, 2023

Conversation

suztomo
Copy link
Member

@suztomo suztomo commented Jun 20, 2023

  • To show 'No change' if there's no change for a library. Before this change, it's blank:

    • Screenshot 2023-06-20 at 10 42 48 PM
    • With this PR, it shows "No change". In practice, there should be at least one google-cloud-shared-dependencies version bump. You wouldn't see this word released.
  • Use proper name in yoshi-code-bot so that it passes CLA.

    • I tested in e539052 and CLA passed.

@suztomo suztomo changed the title fix(ci): yoshi-code-bot account in changelog generation fix(ci): changelog.md generation fix for 'no change' and yoshi-code-bot email Jun 21, 2023
@suztomo suztomo changed the title fix(ci): changelog.md generation fix for 'no change' and yoshi-code-bot email fix(ci): CHANGELOG.md generation fix for 'no change' and yoshi-code-bot email Jun 21, 2023
)
self.assertEqual(entry, f'''## 1.2.3 (2023-06-10)

* No change
Copy link
Member Author

Choose a reason for hiding this comment

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

This is the effect of this pull request.

@@ -22,7 +22,6 @@ jobs:
steps:
- uses: actions/checkout@v3
with:
fetch-depth: 0
Copy link
Member Author

Choose a reason for hiding this comment

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

The fetch-depth default 1 seems enough. No need to fetch all branches.
https://github.com/actions/checkout#checkout-a-different-branch

@@ -40,7 +39,7 @@ jobs:
python3 .github/release-note-generation/split_release_note.py pr_body.txt .
- name: Commit the change and push to the pull request branch
run: |
git config user.email "yoshi-code-bot[bot]@users.noreply.github.com"
git config user.email "[email protected]"
Copy link
Member Author

Choose a reason for hiding this comment

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

This is another fix on this pull request. It was failing CLA check in a3412ef. I confirmed this email address passes the CLA check e539052.

@suztomo suztomo added the automerge Merge the pull request once unit tests and other checks pass. label Jun 21, 2023
@suztomo suztomo added the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 21, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 21, 2023
@suztomo suztomo merged commit 6a5d805 into googleapis:main Jun 21, 2023
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jun 21, 2023
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