Skip to content

Fix release notes diff in github release actions#1882

Open
atandon2024 wants to merge 1 commit intomongodb:mainfrom
atandon2024:fix-github-release-notes
Open

Fix release notes diff in github release actions#1882
atandon2024 wants to merge 1 commit intomongodb:mainfrom
atandon2024:fix-github-release-notes

Conversation

@atandon2024
Copy link
Collaborator

Fix GitHub Actions release notes generation - Corrects an issue where draft release notes included commits from earlier releases due to an incorrect comparison range.

Needs to be tested by triggering Github Actions manually before merging.

@atandon2024 atandon2024 requested a review from a team as a code owner February 6, 2026 19:40
@atandon2024 atandon2024 requested a review from vbabanin February 6, 2026 19:40
Copy link
Member

@rozza rozza left a comment

Choose a reason for hiding this comment

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

One question about fetch depth.

with:
app_id: ${{ vars.APP_ID }}
private_key: ${{ secrets.APP_PRIVATE_KEY }}
fetch-depth: 0
Copy link
Member

Choose a reason for hiding this comment

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

What does this do?

I think fetch-depth: 0 fetches all history for all branches and tags. But is it just for the mongodb-labs/drivers-github-tools/secure-checkout or is it for the java driver repo itself?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@rozza yes fetch-depth 0 fetches all branches and tags.

Per the GitHub Actions documentation (https://docs.github.com/en/actions/tutorials/create-an-example-workflow), drivers-github-tools is only pulled internally to run the action itself. The checkout operates on github.ref.

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