Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions .github/workflows/changelog-preview.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
name: Changelog Preview
on:
pull_request:
types:
- opened
- synchronize
- reopened
- edited
- labeled
- unlabeled
permissions:
contents: write
pull-requests: write

jobs:
changelog-preview:
uses: getsentry/craft/.github/workflows/changelog-preview.yml@v2
secrets: inherit
10 changes: 7 additions & 3 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,15 @@ on:
workflow_dispatch:
inputs:
version:
description: Version to release
required: true
description: Version to release (or "auto")
required: false
force:
description: Force a release even when there are release-blockers (optional)
required: false
permissions:
contents: write
pull-requests: write

jobs:
release:
runs-on: ubuntu-latest
Expand All @@ -25,7 +29,7 @@ jobs:
token: ${{ steps.token.outputs.token }}
fetch-depth: 0
- name: Prepare release
uses: getsentry/action-prepare-release@v1
uses: getsentry/craft@c6e2f04939b6ee67030588afbb5af76b127d8203 # v2
env:
GITHUB_TOKEN: ${{ steps.token.outputs.token }}
Copy link

Choose a reason for hiding this comment

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

Bug: When the optional version input is not provided, an empty string is passed to the release action, which may not be handled as the expected "auto" value.
Severity: MEDIUM

Suggested Fix

Verify if the getsentry/craft action correctly handles an empty string for its version input. If it requires the literal string "auto", consider setting default: 'auto' for the workflow input to ensure the correct value is passed when none is provided.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: .github/workflows/release.yml#L34

Potential issue: The `release.yml` workflow's `version` input was changed to be
optional. When a user triggers this workflow without providing a `version`, GitHub
Actions passes an empty string (`""`) to the `getsentry/craft` action. The input's
description, "(or 'auto')", suggests that the literal string `"auto"` might be the
expected value for automatic versioning. If the `craft` action does not treat an empty
string as equivalent to `"auto"`, the release process may fail when no version is
specified.

Did we get this right? 👍 / 👎 to inform future reviews.

with:
Expand Down