-
Notifications
You must be signed in to change notification settings - Fork 2
chore: fix publish #1296
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: fix publish #1296
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1296 +/- ##
=======================================
Coverage 99.67% 99.67%
=======================================
Files 49 49
Lines 607 607
Branches 35 35
=======================================
Hits 605 605
Misses 2 2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@@ -94,6 +94,9 @@ | |||
"type": "git", | |||
"url": "git@github.com:hypertrace/hyperdash-angular.git" | |||
}, | |||
"publishConfig": { | |||
"registry": "https://us-npm.pkg.dev/hypertrace-repos/npm-publish" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The docs here are wildly confusing. They state
The registry can be configured via the npm environment variable NPM_CONFIG_REGISTRY and will take precedence over the configuration in .npmrc.
which isn't what we saw. Later in the docs, it states:
Notes:
The presence of an .npmrc file will override any specified environment variables.
which is what we saw.
So trying a different approach to split the read and publish registries.
🎉 This PR is included in version 7.1.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
No description provided.