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

Bump minimum ruby version requirement #3818

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

codlp
Copy link

@codlp codlp commented Apr 30, 2024

WHY are these changes introduced?

Fixes #3816

Currently, the Shopify CLI's documentation indicates that, to work with themes, we need to have ruby version 2.7.2 or higher.

However, with ruby version 2.7.2 installed, running the command shopify theme pull --store {store-name} returns an error asking us to install ruby version 2.7.5 or higher.

WHAT is this pull request doing?

Bumping the minimum ruby version requirement in the documentation.

@codlp codlp changed the title Bump minimum ruby version requirement - Issue 3816 Bump minimum ruby version requirement Apr 30, 2024
Copy link
Contributor

@karreiro karreiro left a comment

Choose a reason for hiding this comment

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

Thank you for this PR, @codlp! It looks great to me, I just believe we need to get the CLA check passing :)

@isaacroldan, I've added you as a reviewer because I'm not sure how the lifecycle of the generated_static_pages.json works. I wonder if it can be updated here, or if it should be updated somewhere else and auto-synced in the CLI repository.

@karreiro karreiro requested review from isaacroldan and removed request for jamesmengo May 1, 2024 14:42
@isaacroldan
Copy link
Contributor

@karreiro The docs are still not automated, so after merging this we need to open a PR to the docs.
Also, our CI has a bug and doesn't currently work with remote branches, so you'd need to open a PR yourself to get this merged :/

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.

Update minimum ruby version requirement
3 participants