Skip to content

fix: remove scoped registry override, fix warning message typo [EXT-00]#10734

Merged
Jared Jolton (jjolton-contentful) merged 2 commits intomasterfrom
fix/remove-redundant-registry
Mar 24, 2026
Merged

fix: remove scoped registry override, fix warning message typo [EXT-00]#10734
Jared Jolton (jjolton-contentful) merged 2 commits intomasterfrom
fix/remove-redundant-registry

Conversation

@jjolton-contentful
Copy link
Copy Markdown
Contributor

Purpose

We need to remove the redundant @contentful:registry from .npmrc that is overriding the --registry flag during lerna publish, causing E404. The registry was added to the .npmrc incidentally in #9950 and is safe to remove.

Approach

I included a small typo fix in dam-app-base to ensure it triggers a new release.

Testing steps

Merge and ensure release works.

Breaking Changes

None

Dependencies and/or References

builds on #10733

Deployment

None

Copilot AI review requested due to automatic review settings March 24, 2026 19:19
@jjolton-contentful Jared Jolton (jjolton-contentful) changed the title fix: remove scoped registry override [EXT-00] fix: remove scoped registry override, fix warning message typo [EXT-00] Mar 24, 2026
Copy link
Copy Markdown
Contributor

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

Removes a root-level npm scoped-registry override that was interfering with lerna publish --registry, and includes a minor string typo fix in dam-app-base to trigger a release.

Changes:

  • Removed @contentful:registry override from the root .npmrc so --registry is respected during publish.
  • Fixed a user-facing typo (“incompatibile” → “incompatible”) in dam-app-base.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.npmrc Removes the scoped registry override that was overriding lerna publish --registry.
packages/dam-app-base/src/Editor/Field.tsx Fixes a typo in a warning Note title (also forces a new release).

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

@jjolton-contentful Jared Jolton (jjolton-contentful) merged commit 8932c7a into master Mar 24, 2026
19 checks passed
@jjolton-contentful Jared Jolton (jjolton-contentful) deleted the fix/remove-redundant-registry branch March 24, 2026 19:37
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.

3 participants