Skip to content

fix(ci): 'rc' qualifier ignored when packaging onnxruntime-node#28350

Merged
sanaa-hamel-microsoft merged 1 commit into
mainfrom
dev/sanaahamel/ci-fix-ort-node-pkging-rc
May 5, 2026
Merged

fix(ci): 'rc' qualifier ignored when packaging onnxruntime-node#28350
sanaa-hamel-microsoft merged 1 commit into
mainfrom
dev/sanaahamel/ci-fix-ort-node-pkging-rc

Conversation

@sanaa-hamel-microsoft
Copy link
Copy Markdown
Contributor

@sanaa-hamel-microsoft sanaa-hamel-microsoft commented May 4, 2026

Description

Fix onnxruntime-node and onnxruntime-common NPM packages lacking an RC suffix when built in Release + RC mode.
This isn't great, the suffix looks like -QUAL.DATE-COMMIT. This'll break the publishing pipeline if the packaging pipelines (zip-nuget and NPM) span more than a single day due to same-version checks/enforcement.

Motivation and Context

Missing the RC qualifier/suffix fails the NPM publish pipeline. It correctly assets that the (onnxruntime-node, onnxruntime-common, and onnxruntime-web) do not share a common version specifier.

@sanaa-hamel-microsoft sanaa-hamel-microsoft force-pushed the dev/sanaahamel/ci-fix-ort-node-pkging-rc branch from 754699c to 4890ad0 Compare May 4, 2026 20:39
@sanaa-hamel-microsoft sanaa-hamel-microsoft force-pushed the dev/sanaahamel/ci-fix-ort-node-pkging-rc branch from 4890ad0 to 5becc58 Compare May 5, 2026 15:31
@sanaa-hamel-microsoft
Copy link
Copy Markdown
Contributor Author

@sanaa-hamel-microsoft sanaa-hamel-microsoft merged commit c85ec49 into main May 5, 2026
93 checks passed
@sanaa-hamel-microsoft sanaa-hamel-microsoft deleted the dev/sanaahamel/ci-fix-ort-node-pkging-rc branch May 5, 2026 19:04
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