Skip to content

ci(github): migrate to new NuGet publish workflows and add release-drafter#16

Merged
ncipollina merged 1 commit into
mainfrom
feat/nuget-versioning
Apr 18, 2026
Merged

ci(github): migrate to new NuGet publish workflows and add release-drafter#16
ncipollina merged 1 commit into
mainfrom
feat/nuget-versioning

Conversation

@ncipollina
Copy link
Copy Markdown
Contributor

Summary

  • Replaces build.yaml with publish-preview.yaml (triggers on push to main) and publish-release.yaml (triggers on GitHub release published)
  • Preview NuGet version resolved from Release Drafter's next version — no JSON manifest needed
  • Adds standalone release-drafter.yaml for PR autolabeling and draft release maintenance
  • Adds pr-title-check.yaml for semantic PR title validation

Notes for Reviewers

  • .github/release-drafter.yml was pushed directly to main as a prerequisite (required by Release Drafter to be on the default branch)
  • NUGET_API_KEY secret must be configured in this repo before the publish workflows will work
  • No prereleaseIdentifier override — defaults to preview (stable package)

🤖 Generated with Claude Code

…after

- Remove build.yaml; replace with publish-preview and publish-release
- publish-preview triggers on push to main; version from Release Drafter
- publish-release triggers on GitHub release published event
- Add release-drafter workflow (standalone for PR labeling/draft updates)
- Add pr-title-check workflow
- Update slnx to reflect new workflow files

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ncipollina ncipollina merged commit b5cb392 into main Apr 18, 2026
3 checks passed
@ncipollina ncipollina added the type: ci CI/CD changes label Apr 18, 2026
@ncipollina ncipollina deleted the feat/nuget-versioning branch April 18, 2026 02:28
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 114bfae98f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


jobs:
publish:
uses: LayeredCraft/devops-templates/.github/workflows/publish-preview.yml@main
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Pin reusable workflow refs to immutable versions

This workflow (and the other newly added ones) calls external reusable workflows at @main, which is a mutable ref; because these jobs also use secrets: inherit (including NUGET_API_KEY in publish jobs), any upstream change on LayeredCraft/devops-templates can silently alter release behavior or exfiltrate secrets without a change in this repo. Pin these uses: references to an immutable tag or commit SHA to make CI behavior reproducible and prevent supply-chain drift.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: ci CI/CD changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant