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

feat: STRF-10366 Node 18 Support #1058

Merged
merged 1 commit into from
Jan 31, 2023

Conversation

jairo-bc
Copy link
Contributor

@jairo-bc jairo-bc commented Jan 20, 2023

What?

Node 18 Support

Tickets / Documentation

Screenshots (if appropriate)

Screenshot 2023-01-20 at 10 49 53

cc @bigcommerce/storefront-team

package.json Outdated
@@ -4,8 +4,8 @@
"description": "CLI tool to run BigCommerce Stores locally for theme development.",
"main": "index.js",
"engines": {
"node": ">=14.0.0 <15.0.0",
"npm": ">=v6.11.3 <9.0.0"
"node": ">=17.0.0 <19.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

@jairo-bc your screen says that you've managed to run stencil on Node 14. Is that ok taking into account this line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated

@jairo-bc jairo-bc requested a review from bc-max January 20, 2023 15:50
bc-max
bc-max previously approved these changes Jan 20, 2023
@jairo-bc jairo-bc changed the base branch from master to release-7x January 26, 2023 11:11
@jairo-bc jairo-bc dismissed bc-max’s stale review January 26, 2023 11:11

The base branch was changed.

@jairo-bc jairo-bc requested a review from bc-max January 31, 2023 16:22
@jairo-bc jairo-bc merged commit 88d4176 into bigcommerce:release-7x Jan 31, 2023
jairo-bc added a commit to jairo-bc/stencil-cli that referenced this pull request Mar 30, 2023
jairo-bc added a commit to jairo-bc/stencil-cli that referenced this pull request Mar 30, 2023
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.

2 participants