-
Notifications
You must be signed in to change notification settings - Fork 11
chore: Extract release workflow for automated NuGet package publishing with release-drafter action #179
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
Conversation
…g with release-drafter action
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.
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
Files not reviewed (1)
- .github/workflows/ci.yml: Language not supported
Comments suppressed due to low confidence (1)
.github/workflows/release.yml:10
- [nitpick] Consider renaming 'Create Package nuget' to 'Create NuGet Package' for clearer naming conventions.
+ name: Create Package nuget
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.
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- .github/workflows/ci.yml: Language not supported
Comments suppressed due to low confidence (2)
.github/workflows/release.yml:35
- [nitpick] Consider renaming the step to 'Build NuGet Package' for improved clarity and consistency.
- name: Build package nuget
.github/workflows/release.yml:42
- [nitpick] For consistency with other steps, consider using the GitHub Actions context syntax (e.g. '${{ env.VERSION }}') instead of '$VERSION' in the push command.
- name: Push package
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- .github/workflows/ci.yml: Language not supported
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.
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- .github/workflows/ci.yml: Language not supported
Comments suppressed due to low confidence (1)
.github/workflows/release.yml:43
- [nitpick] Consider quoting the file path that contains the ${{ env.VERSION }} variable to avoid potential issues if the variable ever includes unexpected characters.
run: dotnet nuget push ./output/Netatmo.${{ env.VERSION }}.nupkg -k ${{ secrets.NUGET_ORG_KEY }} -s https://api.nuget.org/v3/index.json
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.
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
Files not reviewed (1)
- .github/workflows/ci.yml: Language not supported
Comments suppressed due to low confidence (1)
.github/workflows/release.yml:18
- The parameter expansion in this line is bash-specific; consider explicitly setting the shell to bash in this step to ensure consistent behavior in the future.
tag=${GITHUB_REF/refs\/tags\/}
This pull request includes significant changes to the GitHub Actions workflows, specifically around the CI and release processes. The most important changes involve the removal of the old CI workflow and the addition of a new release workflow.
Changes to GitHub Actions workflows:
.github/workflows/ci.yml
: Removed the entire CI workflow configuration, which included testing on multiple operating systems, building packages, and publishing artifacts..github/workflows/release.yml
: Added a new release workflow configuration that triggers on tag pushes, builds and tests the package, and publishes the NuGet package and release artifacts.