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

Add webrick as explicit dev dependency #5010

Merged
merged 1 commit into from
Apr 12, 2023

Conversation

kennyadsl
Copy link
Member

Summary

We need webrick to run JS tests using Teaspoon.
This dependency is no more bundled with latest ruby versions and won't be available unless explicitely added to the Gemfile.

Ref: https://www.ruby-lang.org/en/news/2020/12/25/ruby-3-0-0-released/

Not sure why this started to pop now, maybe something with the ruby versions available in CircleCI. Anyway, it seems legit and we should do it.

This is an example of failing build: https://app.circleci.com/pipelines/github/nebulab/solidus/915/workflows/5865af00-dbdd-49f9-8bbb-d057f621a02f/jobs/13055

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed:

  • 📖 I have updated the README to account for my changes.
  • 📑 I have documented new code with YARD.
  • 🛣️ I have opened a PR to update the guides.
  • ✅ I have added automated tests to cover my changes.
  • 📸 I have attached screenshots to demo visual changes.

We need webrick to run JS tests using Teaspoon.
This dependency is no more bundled with latest ruby versions
and won't be available unless explicitely added to the Gemfile.

Ref: https://www.ruby-lang.org/en/news/2020/12/25/ruby-3-0-0-released/

Not sure why this started to pop now, maybe something with the ruby
versions available in CircleCI. Anyway, it seems legit and we should do
it.
@kennyadsl kennyadsl added the type:bug Error, flaw or fault label Apr 12, 2023
@kennyadsl kennyadsl self-assigned this Apr 12, 2023
@kennyadsl kennyadsl requested a review from a team as a code owner April 12, 2023 07:02
@github-actions github-actions bot added the changelog:repository Changes to the repository not within any gem label Apr 12, 2023
Copy link
Contributor

@waiting-for-dev waiting-for-dev left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Contributor

@rainerdema rainerdema left a comment

Choose a reason for hiding this comment

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

🚢

@kennyadsl kennyadsl merged commit 482a56a into solidusio:master Apr 12, 2023
@kennyadsl kennyadsl deleted the kennyadsl/fix-teaspoon branch April 12, 2023 08:50
@kennyadsl kennyadsl added backport-v3.2 backport-v3.3 Backport this pull-request to v3.3 labels Apr 17, 2023
@github-actions
Copy link

💔 Some backports could not be created

Status Branch Result
v3.2
v3.3 An unhandled error occurred. Please see the logs for details

Manual backport

To create the backport manually run:

backport --pr 5010

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@github-actions
Copy link

💚 All backports created successfully

Status Branch Result
v3.2
v3.3

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-v3.3 Backport this pull-request to v3.3 changelog:repository Changes to the repository not within any gem type:bug Error, flaw or fault
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants