-
-
Notifications
You must be signed in to change notification settings - Fork 237
docs: add npmx-redirect extension to readme #1349
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
📝 WalkthroughWalkthroughThe README.md documentation has been updated to expand guidance on browser extensions. The change replaces a single reference to the npmx-replace extension with broader coverage that now includes both npmx-replace and npmx-redirect extensions, detailing Chrome and Firefox platform availability with direct links. A related entry for npmx-redirect is also added to the Related projects section. The diff contains unresolved merge conflict markers. No public API signatures have been modified. Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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.
Actionable comments posted: 1
| > Want automatic redirects? Try the [npmx-replace browser extension](https://github.com/tylersayshi/npmx-replace-extension) (Chrome only for now). | ||
| > Want automatic redirects? Try the [npmx-replace browser extension](https://github.com/tylersayshi/npmx-replace-extension) (Chrome only for now) or the separate [npmx-redirect extension](https://github.com/iaverages/npmx-redirect) for [Chrome](https://chromewebstore.google.com/detail/lbhjgfgpnlihfmobnohoipeljollhlnb) / [Firefox](https://addons.mozilla.org/en-GB/firefox/addon/npmx-redirect/). | ||
| > | ||
| > > > > > > > conflict 1 of 1 ends |
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.
Critical: Unresolved merge conflict marker must be removed.
Line 121 contains a merge conflict marker (> > > > > > > conflict 1 of 1 ends) that will break the markdown rendering and prevent this PR from being merged.
🔧 Proposed fix to remove the conflict marker
> Want automatic redirects? Try the [npmx-replace browser extension](https://github.com/tylersayshi/npmx-replace-extension) (Chrome only for now) or the separate [npmx-redirect extension](https://github.com/iaverages/npmx-redirect) for [Chrome](https://chromewebstore.google.com/detail/lbhjgfgpnlihfmobnohoipeljollhlnb) / [Firefox](https://addons.mozilla.org/en-GB/firefox/addon/npmx-redirect/).
->
-> > > > > > > conflict 1 of 1 ends|
I have no idea what I did to my local repo, also messed up the merge conflict, I will remake this PR :thinking. |
Adds an alternative browser extension for automatic redirects. Published on both chrome web store and firefox add-ons.