Skip to content

Add github actions#16

Merged
1fabi0 merged 7 commits intomainfrom
create-workflows
Apr 15, 2026
Merged

Add github actions#16
1fabi0 merged 7 commits intomainfrom
create-workflows

Conversation

@1fabi0
Copy link
Copy Markdown
Member

@1fabi0 1fabi0 commented Apr 14, 2026

refs: #5 #6

Copilot AI review requested due to automatic review settings April 14, 2026 11:16
Copy link
Copy Markdown

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.

Pull request overview

Adds GitHub Actions workflows to automate CI validation, security scanning, and releases (including NuGet publishing), along with NuGet package metadata updates (readme/icon) to support the first release process.

Changes:

  • Add PR CI workflow to restore/build/test and report unit test results.
  • Add release workflow that generates versions from Conventional Commits, creates GitHub releases, and calls a reusable NuGet publish workflow.
  • Add Trivy filesystem scan workflow with SARIF upload; update NuGet package metadata/readme/icon assets.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/ShinyPDF/ShinyPDF.csproj Adds NuGet readme/icon metadata and tweaks package description/tags.
src/ShinyPDF/Resources/PackageReadme.md Updates package readme header/logo image reference.
src/ShinyPDF/Resources/Description.md Removes old description markdown content.
.github/workflows/release.yaml New workflow to generate changelog/version and create GitHub releases, then trigger NuGet publish.
.github/workflows/publish.yaml New reusable workflow to build/pack and push package to NuGet.
.github/workflows/pull-request.yaml New PR validation workflow running restore/build/tests with test reporting.
.github/workflows/code-scan.yaml New Trivy scan workflow uploading SARIF results to GitHub Security.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/pull-request.yaml Outdated
Comment thread src/ShinyPDF/ShinyPDF.csproj Outdated
Comment thread src/ShinyPDF/Resources/PackageReadme.md Outdated
Comment thread .github/workflows/pull-request.yaml
Comment thread .github/workflows/publish.yaml Outdated
Comment thread .github/workflows/publish.yaml Outdated
Copy link
Copy Markdown

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.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/pull-request.yaml
Comment thread .github/workflows/publish.yaml Outdated
Copy link
Copy Markdown

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.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/ShinyPDF/ShinyPDF.csproj Outdated
Comment thread .github/workflows/pull-request.yaml Outdated
Comment thread .github/workflows/code-scan.yaml
Copilot AI review requested due to automatic review settings April 15, 2026 09:51
Copy link
Copy Markdown

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.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/release.yaml
Comment thread .github/workflows/publish.yaml
@1fabi0 1fabi0 merged commit ee90cd2 into main Apr 15, 2026
3 checks passed
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