-
Notifications
You must be signed in to change notification settings - Fork 1
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
unpin digests, autoclose renovate PRs #75
Changes from all commits
99d435c
b92e48a
9007fdd
3a8d323
37acfb3
8ac24c6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -33,7 +33,7 @@ jobs: | |||||||
echo "GITHUB_WORKSPACE=$GITHUB_WORKSPACE" >> $GITHUB_ENV | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. suggestion: Use built-in GitHub context for workspace path Instead of setting the GITHUB_WORKSPACE as an environment variable, consider using the built-in ${{ github.workspace }} context variable directly. This ensures consistency and avoids potential issues with environment variable scope.
Suggested change
|
||||||||
- name: download artifact from previous run | ||||||||
id: download-artifact | ||||||||
uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4 | ||||||||
uses: actions/download-artifact@v4 # v4 | ||||||||
with: | ||||||||
name: ${{ inputs.artifact-name }} | ||||||||
path: ${{ env.GITHUB_WORKSPACE }}/dist | ||||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: Consider simplifying bump type determination logic
The current if-else structure for determining the bump type could be simplified using a case statement or a mapping. This would make the code more concise and easier to maintain, especially if more bump types are added in the future.