Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Rename bin/webpack[-dev-server] to bin/webpacker[-dev-server]#3252

Merged
guillaumebriday merged 1 commit into
rails:masterfrom
shakacode:justin808-rename-bins
Jan 6, 2022
Merged

Rename bin/webpack[-dev-server] to bin/webpacker[-dev-server]#3252
guillaumebriday merged 1 commit into
rails:masterfrom
shakacode:justin808-rename-bins

Conversation

@justin808
Copy link
Copy Markdown
Contributor

@justin808 justin808 commented Dec 29, 2021

Renamed /bin/webpack to /bin/webpacker and /bin/webpack-dev-server
to bin/webpacker-dev-server to avoid confusion with underlying webpack
executables.

Addresses #3237.

Renamed `/bin/webpack` to `/bin/webpacker` and `/bin/webpack-dev-server`
to `bin/webpacker-dev-server` to avoid confusion with underlying webpack
executables.
@justin808 justin808 self-assigned this Dec 29, 2021
@justin808 justin808 added this to the 6.0 milestone Dec 29, 2021
@justin808 justin808 changed the title bin/webpack[-dev-server] to webpacker[-dev-server] Rename bin/webpack[-dev-server] to bin/webpacker[-dev-server] Dec 29, 2021
@justin808
Copy link
Copy Markdown
Contributor Author

CI passes. What else should be tested? Most changes are in the docs.

Comment thread docs/v6_upgrade.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think this should be the 20th step, the last step should always be Try your app! 😀

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We've got some other PRs with changes here, so I'll clean up all at once!

Comment thread docs/v6_upgrade.md Outdated
Copy link
Copy Markdown

@frullah frullah Jan 1, 2022

Choose a reason for hiding this comment

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

Suggested change
21. Update any scripts that called `/bin/webpack` or `bin/webpack-dev-server` to `/bin/webpacker` or `bin/webpacker-dev-server`
21. Update any scripts that called `bin/webpack` to `bin/webpacker` and `bin/webpack-dev-server` to `bin/webpacker-dev-server`

i think it can reduce confusion

@guillaumebriday guillaumebriday merged commit 6614a4f into rails:master Jan 6, 2022
@guillaumebriday
Copy link
Copy Markdown
Member

Nice!

@justin808 justin808 deleted the justin808-rename-bins branch January 6, 2022 09:18
tricknotes added a commit to tricknotes/webpacker that referenced this pull request Jan 25, 2022
These commands have been renamed in rails#3252.
justin808 pushed a commit to shakacode/shakapacker that referenced this pull request Jan 25, 2022
These commands have been renamed in rails/webpacker#3252.
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.

4 participants