Skip to content
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

Move all npm packages to @rails scope #34905

Merged
merged 1 commit into from
Jan 10, 2019

Conversation

javan
Copy link
Contributor

@javan javan commented Jan 9, 2019

Fixes #33083

actioncable   → @rails/actioncable
actiontext    → @rails/actiontext
activestorage → @rails/activestorage
rails-ujs     → @rails/ujs

TODO:

*npm doesn’t have a built-in system for moving or renaming packages

Copy link
Contributor

@kaspth kaspth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! What happens when people next go to call yarn install and the packages have been renamed?

RELEASING_RAILS.md Outdated Show resolved Hide resolved
@javan
Copy link
Contributor Author

javan commented Jan 9, 2019

What happens when people next go to call yarn install and the packages have been renamed?

The packages aren't being renamed, technically. They're being published as new packages with new names. So, existing applications will need to update their package.jsons to point to the new packages.

@javan javan force-pushed the move-npm-packages-to-rails-scope branch 3 times, most recently from e096280 to e2d4d04 Compare January 10, 2019 15:36
@javan javan force-pushed the move-npm-packages-to-rails-scope branch from e2d4d04 to 86b489e Compare January 10, 2019 16:02
@javan javan merged commit 2163762 into rails:master Jan 10, 2019
@javan javan deleted the move-npm-packages-to-rails-scope branch January 10, 2019 16:50
@connorshea
Copy link
Contributor

@javan shouldn't this be mentioned in the CHANGELOG and/or Release Notes?

@javan
Copy link
Contributor Author

javan commented Jan 28, 2019

@connorshea, yes thank you! Added here e3d4333.

@benreyn
Copy link

benreyn commented Nov 15, 2019

I just ran into this issue on a project where I couldn't figure out why I wasn't getting this fix and there was no version 6.0.0 for rails-ujs. Does npm provide any mechanism for communicating this better? Other than digging through the changelog, issues, and source code, there was no (obvious) indication of this change. At the very least should a post-install script log a message that says something to the effect of rails-ujs has been deprecated. Please use @rails/ujs instead (This would involve a version bump to the old package)

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

Successfully merging this pull request may close these issues.

Collect all Rails npm packages under @rails/*
4 participants