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

Bundle code into a single file without dependencies #393

Merged
merged 15 commits into from
Sep 30, 2021
Merged

Conversation

ndelangen
Copy link
Member

@ndelangen ndelangen commented Aug 25, 2021

  • Bundle the chromatic-cli bin and action, so they are a single file without dependencies!
    No more dependencies need to be installed!
    This should increase
  • Add proper support for yarn berry with pnp enabled
    by asking the yarn cli for storybook packages,
    instead of trying to resolve them all.
    (use the resolving mechanism is asking the yarn cli fails)

This fixes: #166

  • Add case handling workflow_run for our action

This fixes: #396

@ndelangen ndelangen self-assigned this Aug 25, 2021
@ndelangen
Copy link
Member Author

@ghengeveld I think it's ready for review!

@ndelangen ndelangen added dependencies Classification: Update one or more dependencies version enhancement Classification: New feature or request labels Aug 26, 2021
OzzieOrca added a commit to CruGlobal/mpdx-react that referenced this pull request Aug 31, 2021
@ghengeveld ghengeveld changed the title Tech/bundling Bundle code into a single file without dependencies Sep 17, 2021
@ndelangen
Copy link
Member Author

I'm, testing this across a few repositories..

@ndelangen
Copy link
Member Author

looking good now:
https://github.com/chromaui/chromatic/pull/5475

@ndelangen ndelangen force-pushed the tech/bundling branch 3 times, most recently from 68c1672 to e11522d Compare September 23, 2021 09:08
@ndelangen
Copy link
Member Author

I took out the breaking change, so it should be backwards compatible, I tested this on the chromatic repo

@ndelangen ndelangen merged commit c372e18 into next Sep 30, 2021
@ndelangen ndelangen deleted the tech/bundling branch September 30, 2021 12:42
@ndelangen ndelangen restored the tech/bundling branch September 30, 2021 13:12
OzzieOrca added a commit to CruGlobal/mpdx-react that referenced this pull request Oct 1, 2021
- chromaui/chromatic-cli#393 was merged and released, adding support for Yarn v3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Classification: Update one or more dependencies version enhancement Classification: New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider adding support for workflow_run event in GitHub Action Yarn Berry Support
1 participant