Skip to content

Conversation

@dm3ch
Copy link
Contributor

@dm3ch dm3ch commented Nov 25, 2025

Change Summary

What does this PR change?
Introduce pipelines for lib publishing

Related Issue/Ticket:
ITSM-1219

Testing & Verification

How was this tested?

  • Unit tests
  • Integration tests
  • Manual testing (describe steps)
  • Verified on staging

Risk Assessment

Risk Level:

  • Low - Minor changes, no operational impact
  • Medium - Moderate changes, limited impact, standard rollback available
  • High - Significant changes, potential operational impact, complex rollback

Risks & Impact

@dm3ch dm3ch marked this pull request as draft November 25, 2025 14:27
@dm3ch dm3ch force-pushed the chore/finish-pipelines branch from 46f306a to c97aeaf Compare November 25, 2025 15:06
CI/CD Bot and others added 6 commits November 25, 2025 15:07
@dm3ch
Copy link
Contributor Author

dm3ch commented Nov 25, 2025

Don't forget to cleanup:

  • Testing lib versions both github / npmjs
  • Drop tags and releases
  • Check security considerations
  • Review

dm3ch and others added 11 commits November 25, 2025 17:38
- project: sdk-core 0.1.1-pipeline-test.4
- project: sdk-core 0.1.1-pipeline-test.5
- project: sdk-core 0.1.1-pipeline-test.6
- project: sdk-core 0.1.1-pipeline-test.7
- project: sdk-core 0.1.1-pipeline-test.8
dm3ch and others added 7 commits November 25, 2025 17:52
- project: sdk-core 0.1.1-pipeline-test.9
- project: sdk-core 0.1.1-pipeline-test.10
- project: sdk-core 0.1.1-pipeline-test.11
@dm3ch dm3ch requested review from Copilot and vbrvk and removed request for Copilot November 25, 2025 16:01
@dm3ch dm3ch marked this pull request as ready for review November 25, 2025 16:01
Copy link

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

This PR overhauls the SDK release and publishing pipelines to use NX's built-in release tooling. The workflow consolidates version bumping, changelog generation, and publishing into a single unified pipeline that supports both npmjs and GitHub Packages registries.

Key changes:

  • Migrates from custom version bumping to NX release with configurable semver bumps and prerelease support
  • Consolidates release and publish workflows into a single pipeline with separate jobs for npmjs and GitHub Packages
  • Adds dry-run capability for testing releases without publishing

Reviewed changes

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

Show a summary per file
File Description
.github/workflows/release.yml Complete rewrite to use NX release tooling with three jobs: release, publish-npmjs, and publish-github-packages
.github/workflows/publish.yml Removed standalone publish workflow (now integrated into release.yml)
.github/workflows/test-publish.yml Removed dry-run testing workflow (dry-run now built into release.yml)
nx.json Added version, git, and releaseTag configuration for NX release
typescript/README.md Updated release process documentation to reflect new workflow

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

dm3ch and others added 2 commits November 25, 2025 18:05
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@dm3ch dm3ch requested a review from Copilot November 25, 2025 16:06
Copy link

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 5 out of 5 changed files in this pull request and generated 2 comments.


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

CI/CD Bot and others added 2 commits November 25, 2025 16:07
- project: sdk-core 0.1.1-pipeline-final-test.0
@vbrvk vbrvk merged commit 9a6517a into master Nov 26, 2025
1 check passed
@vbrvk vbrvk deleted the chore/finish-pipelines branch November 26, 2025 14:56
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