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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.x] Yarn v3 for stable webpacker 5.x gem #3098

Merged
merged 11 commits into from Aug 17, 2021

Conversation

pedrofurtado
Copy link
Member

@pedrofurtado pedrofurtado commented Jul 26, 2021

Yarn v3 is released! Webpacker is already ready for this 馃帀

@@ -29,14 +29,6 @@ jobs:
gemfile: gemfiles/Gemfile-rails.6.0.x
experimental: [false]
include:
- ruby: 2.5
Copy link
Member

Choose a reason for hiding this comment

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

Why did you remove these versions?

Copy link
Member Author

Choose a reason for hiding this comment

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

@guillaumebriday Because Rails 7.0.0-alpha (edge) does not support ruby 2.5 and 2.6, only 2.7+ 馃憤

Choose a reason for hiding this comment

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

I think so these removes can be done in other PR separately, only that change context to rails 7. What do you think about that?

Copy link
Member

Choose a reason for hiding this comment

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

Yes I think it's should not be in this PR

Copy link
Member Author

Choose a reason for hiding this comment

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

Considering that branch 5-x-stable does not receive frequent changes, and that this PR is small enough to be understood well (despite CI fixes be together with adding of Yarn v3), can we merge it? 馃嵒

Copy link

Choose a reason for hiding this comment

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

@pedrofurtado sounds like the preferred way forward is for you to pull the workflow changes out of this PR, focus it 馃挴 on just bringing yarn v3 support in for a quick merge....

Copy link
Member Author

Choose a reason for hiding this comment

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

PR splitted in two: this that provides yarn 3 support, and other that fixes CI ( #3106 )

@imajes
Copy link

imajes commented Jul 29, 2021

any chance we can get this merged and released soon? we rely on a yarn package that needs yarn 3, and so webpacker is continually complaining :(

@pedrofurtado
Copy link
Member Author

pedrofurtado commented Aug 4, 2021

@imajes For a while, in order to not block your team on Yarn upgrading, please use this fork:

# Gemfile
gem 'webpacker', github: 'pedrofurtado/webpacker', branch: 'v5.4.0-with-yarnv3'

# package.json
{
  "dependencies": {
    "@pedrofurtado/webpacker": "5.4.0-patched2"
  }
}

We here are using this fork (maintained by me until merge of this PR), while a new official 5.x release is not rolled out 馃 馃

@imajes
Copy link

imajes commented Aug 4, 2021

@pedrofurtado i have already been using your branch, alas. I was hoping for something a bit more official... especially from this team :/

@pedrofurtado
Copy link
Member Author

@guillaumebriday Are we ready for merge? 馃帀

@dhh dhh merged commit 8fa4f22 into rails:5-x-stable Aug 17, 2021
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.

None yet

5 participants