Skip to content

ci: remove publish.yml as this is handled by semantic release - #1446

Merged
madhavilosetty-intel merged 2 commits into
mainfrom
ciFix
Sep 8, 2025
Merged

ci: remove publish.yml as this is handled by semantic release#1446
madhavilosetty-intel merged 2 commits into
mainfrom
ciFix

Conversation

@rsdmike

@rsdmike rsdmike commented Aug 20, 2025

Copy link
Copy Markdown
Member

PR Checklist

  • Unit Tests have been added for new changes
  • API tests have been updated if applicable
  • All commented code has been removed
  • If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?

If the there are associated PRs in other repositories, please link them here (i.e. device-management-toolkit/repo#365 )

Copilot AI review requested due to automatic review settings August 20, 2025 22:53

Copilot AI left a comment

Copy link
Copy Markdown

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 removes the manual NPM package publishing workflow as it's now automated through semantic release. This eliminates the need for manual release management and ensures consistent, automated publishing.

  • Removes the entire .github/workflows/publish.yml file
  • Transitions from manual to automated release management via semantic release

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

graikhel-intel
graikhel-intel previously approved these changes Aug 20, 2025
@madhavilosetty-intel
madhavilosetty-intel merged commit 999eb58 into main Sep 8, 2025
4 checks passed
@RosieAMT

Copy link
Copy Markdown

🎉 This PR is included in version 3.3.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants