8000 chore: Extract release workflow for automated NuGet package publishing with release-drafter action by Riges · Pull Request #179 · Riges/Netatmo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 7 commits into from
Apr 14, 2025

Conversation

Riges
Copy link
Owner
@Riges Riges commented Apr 11, 2025

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.

@Copilot Copilot AI review requested due to automatic review settings April 11, 2025 12:37
Copy link
Contributor
@Copilot Copilot AI left a 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

@Riges Riges requested a review from Copilot April 11, 2025 12:40
Copy link
Contributor
@Copilot Copilot AI left a 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

Riges and others added 2 commits April 11, 2025 14:47
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Riges Riges requested a review from Copilot April 11, 2025 12:49
Copy link
Contributor
@Copilot Copilot AI left a 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

@Riges Riges requested a review from Copilot April 11, 2025 12:51
Copy link
Contributor
@Copilot Copilot AI left a 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

@Riges Riges requested a review from Copilot April 11, 2025 12:54
Copy link
Contributor
@Copilot Copilot AI left a 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\/}

@Riges Riges merged commit f1dcd79 into master Apr 14, 2025
6 checks passed
@Riges Riges deleted the feature/publish_release_refactoring branch April 14, 2025 08:51
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.

1 participant
0