Skip to content

Conversation

@BPScott
Copy link
Member

@BPScott BPScott commented Sep 13, 2021

WHY are these changes introduced?

sewing-kit-next has been renamed to loom, and has been moved into the shopify npm namespace. Code behaviour in the latest packages from the @sewing-kit namespace is identical to the corresponding package in the @shopify namespace aside from the binary name, the config file name and the metadata directory.

WHAT is this pull request doing?

  • Replaces @sewing-kit/core with @shopify/loom, @sewing-kit/cli with @shopify/loom-cli and @sewing-kit/plugin-* with @shopify/loom-plugin-*
  • Use the loom executable instead of skn
  • Rename sewing-kit.config.ts to loom.config.ts
  • Ignore .loom instead of .sewing-kit in ignore files

How to 🎩

  • Do a build on the main branch and then move it to a temporary place with mv build build-og
  • Checkout this branch run yarn to update dependencies, run a yarn clean to delete the build folder
  • Run yarn build to generate a new build
  • Run diff -ru -x '*.tsbuildinfo' build-og build to see that the contents of the old and new build folders are identical.

@BPScott BPScott requested a review from dahukish September 13, 2021 23:00
sewing-kit-next has been renamed to loom, and has been moved into the
shopify npm namespace. Code behaviour is identical aside from the
package names, the config file name and the metadata directory.
@BPScott BPScott added the 🤖Skip Changelog Causes CI to ignore changelog update check. label Sep 13, 2021
@github-actions
Copy link
Contributor

size-limit report

Path Size
cjs 163.35 KB (0%)
esm 96.08 KB (0%)
esnext 142.98 KB (0%)
css 34.52 KB (0%)

Copy link

@dahukish dahukish left a comment

Choose a reason for hiding this comment

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

:shipit:

@BPScott BPScott merged commit 5c2230a into main Sep 13, 2021
@BPScott BPScott deleted the loom branch September 13, 2021 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖Skip Changelog Causes CI to ignore changelog update check.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants