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

Create H2 Shopify/Cli plugin #194

Merged
merged 8 commits into from
Nov 22, 2022
Merged

Create H2 Shopify/Cli plugin #194

merged 8 commits into from
Nov 22, 2022

Conversation

cartogram
Copy link
Contributor

@cartogram cartogram commented Nov 15, 2022

This PR converts the CLI into a oclif plugin for the Shopify CLI. I had to do some magic with a post-install script and convert the CLI to ESM.

We will need to change the way that the CLI is run in the demo store and tests. Related to #199 @frandiox interested in your thoughts here.

@github-actions github-actions bot had a problem deploying to preview November 15, 2022 15:52 Failure
@github-actions github-actions bot had a problem deploying to preview November 16, 2022 07:44 Failure
@github-actions github-actions bot had a problem deploying to preview November 16, 2022 16:30 Failure
@github-actions github-actions bot had a problem deploying to preview November 16, 2022 16:52 Failure
@cartogram cartogram marked this pull request as ready for review November 16, 2022 17:22
@cartogram cartogram requested a review from a team November 16, 2022 17:22
Copy link
Contributor

@blittle blittle left a comment

Choose a reason for hiding this comment

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

I think it's a good idea to align on the shopify cli with oclif. I think part of the issues we are having with ESM is none of our packages are compiling. Everything expects to run in ts-node. I think we need to convert each of the packages to actually compile an output, and properly define an exports property within each package.json. Ideally we shouldn't use tsc for that compilation.

I pulled this down and ran it locally. I can't get anything to build or run. Work in progress I assume?

Update: @frandiox for the win! #202

"sourceMap": true,
"inlineSources": true,
"noUncheckedIndexedAccess": true,
"outDir": "dist",
Copy link
Contributor

Choose a reason for hiding this comment

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

I wish we wouldn't use tsc for compiling :(

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, I just did this temporarily because @frandiox has been working on this separately.

@cartogram
Copy link
Contributor Author

@blittle moving this back to draft until I can sort out the CI errors and wait until the turpo repo stuff merges.

@cartogram cartogram marked this pull request as draft November 17, 2022 17:20
@frandiox
Copy link
Contributor

I think the CI errors here are similar to what I'm getting in #199
I think it's related to NPM workspaces working differently on CI for some reason. I think probably the binaries are at the top node_modules/.bin and for some reason they can't be found from packages/templates child directories. Might be wrong tho.

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

3 participants