refactor: use swc pack instead of ncc #140
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
byCedric
force-pushed
the
main
branch
2 times, most recently
from
2f073f5
to
414c169
Compare
byCedric
force-pushed
the
refactor/ncc-sucrace
branch
3 times, most recently
from
282fa55
to
636e416
Compare
byCedric
force-pushed
the
refactor/ncc-sucrace
branch
from
79ae09c
to
fd997b4
Compare
Unfortunately, it seems that we cant disable this in swc pack
byCedric
force-pushed
the
refactor/ncc-sucrace
branch
from
28312c8
to
43bc9fa
Compare
byCedric
added a commit
to byCedric/expo-github-action
that referenced
this pull request
Jan 12, 2022
* refactor: implement swc instead of ncc * chore: tweak the swc implementation with externals * chore: remove ncc from the project * fix: update the entrypoint in the main action file * chore: fix linting issues * chore: add swc folder to ignore list * chore: update gitignore list * fix: force-remove sourcemaps from the build Unfortunately, it seems that we cant disable this in swc pack
github-actions bot
pushed a commit
that referenced
this pull request
Jan 13, 2022
## [7.0.0](6.0.0...7.0.0) (2022-01-13) ### ⚠ BREAKING CHANGES * some input options are deprecated - Username and password authentication is dropped in favor of tokens - Custom cache keys are dropped - Custom install error handler is dropped ### Bug fixes * use which instead of npx to authenticate ([#147](#147)) ([c5d2c0f](c5d2c0f)) ### Code changes * allow multiple entrypoints for different sub-actions ([#121](#121)) ([8a7306a](8a7306a)) * clean up setup action ([#146](#146)) ([439ff4c](439ff4c)) * enable caches by default ([#148](#148)) ([72d4067](72d4067)) * replace libnpm with npm cli ([#139](#139)) ([4f28493](4f28493)) * roll back to ncc because of swc bundling issues ([#144](#144)) ([3007741](3007741)) * update the outdated workflows and use node 16 ([#138](#138)) ([3134e26](3134e26)) * upgrade to node 16 github actions runtime ([#137](#137)) ([3bf9181](3bf9181)) * use boolean input instead of manual conversion ([#127](#127)) ([f1fe877](f1fe877)) * use spaces for indentation ([#126](#126)) ([ff2ff0f](ff2ff0f)) * use swc pack instead of ncc ([#140](#140)) ([3f9e947](3f9e947)) ### Other chores * add basic e2e test workflow for main action ([#141](#141)) ([414c169](414c169)) * add manual trigger to the tests ([4908a48](4908a48)) * always use lf for build files ([#122](#122)) ([a9b9cfe](a9b9cfe)) * bump semver-regex from 3.1.2 to 3.1.3 ([#114](#114)) ([50c0ef9](50c0ef9)) * bump tar from 4.4.13 to 4.4.19 ([#112](#112)) ([5f5673b](5f5673b)) * bump tmpl from 1.0.4 to 1.0.5 ([#115](#115)) ([92ec93f](92ec93f)) * clean up dependencies and eslint rules ([#145](#145)) ([3ae7999](3ae7999)) * clean up master references in release and workflows ([#104](#104)) ([a2d9b1c](a2d9b1c)) * clean up tsconfig and eslint rules ([#123](#123)) ([c20146f](c20146f)) * lint root files with prettier and eslint ([#125](#125)) ([1cfa7bb](1cfa7bb)) * rebuild project ([45b79a0](45b79a0)) * rebuild project after dependency bumps ([b099f17](b099f17)) * remove husky and improve ci build message ([#128](#128)) ([39b0d36](39b0d36)) * simplify review and update build detection ([#142](#142)) ([ee415b0](ee415b0)) * update `setup-node` action to `v2` release ([#119](#119)) ([3b3e6b8](3b3e6b8)) * update the action manifest ([633cdf0](633cdf0)) * upgrade dev dependencies and rebuild action ([#120](#120)) ([186ef00](186ef00)) ### Documentation changes * fix broken sup element in readme ([0b1822a](0b1822a)) * fix github capitalization ([#116](#116)) ([b602469](b602469)) * fix unsplash comment on pr branch reference ([#110](#110)) ([f14253e](f14253e)) * update expo.io links to expo.dev ([#111](#111)) ([1c8e5a2](1c8e5a2)) * update node version reference in readme ([#107](#107)) ([3ba667b](3ba667b)) * update readme and contributing guides ([521de72](521de72)) * use `main` branch to update `v{major}` tag ([#103](#103)) ([def44e2](def44e2))
github-actions bot
pushed a commit
that referenced
this pull request
Jan 15, 2022
## [7.0.0](6.0.0...7.0.0) (2022-01-15) ### ⚠ BREAKING CHANGES * some input options are deprecated - Username and password authentication is dropped in favor of tokens - Custom cache keys are dropped - Custom install error handler is dropped ### New features * add comment qr code subaction ([#149](#149)) ([4648212](4648212)) ### Bug fixes * use which instead of npx to authenticate ([#147](#147)) ([c5d2c0f](c5d2c0f)) ### Code changes * allow multiple entrypoints for different sub-actions ([#121](#121)) ([8a7306a](8a7306a)) * clean up setup action ([#146](#146)) ([439ff4c](439ff4c)) * enable caches by default ([#148](#148)) ([72d4067](72d4067)) * replace libnpm with npm cli ([#139](#139)) ([4f28493](4f28493)) * roll back to ncc because of swc bundling issues ([#144](#144)) ([3007741](3007741)) * update the outdated workflows and use node 16 ([#138](#138)) ([3134e26](3134e26)) * upgrade to node 16 github actions runtime ([#137](#137)) ([3bf9181](3bf9181)) * use assert instead of if statements ([#150](#150)) ([e4ef6e6](e4ef6e6)) * use boolean input instead of manual conversion ([#127](#127)) ([f1fe877](f1fe877)) * use spaces for indentation ([#126](#126)) ([ff2ff0f](ff2ff0f)) * use swc pack instead of ncc ([#140](#140)) ([3f9e947](3f9e947)) ### Other chores * add basic e2e test workflow for main action ([#141](#141)) ([414c169](414c169)) * add manual trigger to the tests ([4908a48](4908a48)) * always use lf for build files ([#122](#122)) ([a9b9cfe](a9b9cfe)) * bump semver-regex from 3.1.2 to 3.1.3 ([#114](#114)) ([50c0ef9](50c0ef9)) * bump tar from 4.4.13 to 4.4.19 ([#112](#112)) ([5f5673b](5f5673b)) * bump tmpl from 1.0.4 to 1.0.5 ([#115](#115)) ([92ec93f](92ec93f)) * clean up dependencies and eslint rules ([#145](#145)) ([3ae7999](3ae7999)) * clean up master references in release and workflows ([#104](#104)) ([a2d9b1c](a2d9b1c)) * clean up tsconfig and eslint rules ([#123](#123)) ([c20146f](c20146f)) * lint root files with prettier and eslint ([#125](#125)) ([1cfa7bb](1cfa7bb)) * rebuild project ([45b79a0](45b79a0)) * rebuild project after dependency bumps ([b099f17](b099f17)) * remove husky and improve ci build message ([#128](#128)) ([39b0d36](39b0d36)) * simplify review and update build detection ([#142](#142)) ([ee415b0](ee415b0)) * update `setup-node` action to `v2` release ([#119](#119)) ([3b3e6b8](3b3e6b8)) * update semantic release to update the build ([354dc08](354dc08)) * update the action manifest ([633cdf0](633cdf0)) * upgrade dev dependencies and rebuild action ([#120](#120)) ([186ef00](186ef00)) ### Documentation changes * fix broken sup element in readme ([0b1822a](0b1822a)) * fix github capitalization ([#116](#116)) ([b602469](b602469)) * fix unsplash comment on pr branch reference ([#110](#110)) ([f14253e](f14253e)) * update default values in readme ([c2f11d2](c2f11d2)) * update expo.io links to expo.dev ([#111](#111)) ([1c8e5a2](1c8e5a2)) * update node version reference in readme ([#107](#107)) ([3ba667b](3ba667b)) * update readme and contributing guides ([521de72](521de72)) * use `main` branch to update `v{major}` tag ([#103](#103)) ([def44e2](def44e2))
|
🎉 This PR is included in version 7.0.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
github-actions bot
pushed a commit
that referenced
this pull request
Jan 17, 2022
## [7.0.0](6.0.0...7.0.0) (2022-01-17) ### ⚠ BREAKING CHANGES * some input options are deprecated - Username and password authentication is dropped in favor of tokens - Custom cache keys are dropped - Custom install error handler is dropped ### New features * add comment qr code subaction ([#149](#149)) ([4648212](4648212)) ### Bug fixes * remove double resolving package versions ([#152](#152)) ([d53d38f](d53d38f)) * use which instead of npx to authenticate ([#147](#147)) ([c5d2c0f](c5d2c0f)) ### Code changes * allow multiple entrypoints for different sub-actions ([#121](#121)) ([8a7306a](8a7306a)) * clean up setup action ([#146](#146)) ([439ff4c](439ff4c)) * enable caches by default ([#148](#148)) ([72d4067](72d4067)) * replace libnpm with npm cli ([#139](#139)) ([4f28493](4f28493)) * roll back to ncc because of swc bundling issues ([#144](#144)) ([3007741](3007741)) * update the outdated workflows and use node 16 ([#138](#138)) ([3134e26](3134e26)) * upgrade to node 16 github actions runtime ([#137](#137)) ([3bf9181](3bf9181)) * use assert instead of if statements ([#150](#150)) ([e4ef6e6](e4ef6e6)) * use boolean input instead of manual conversion ([#127](#127)) ([f1fe877](f1fe877)) * use spaces for indentation ([#126](#126)) ([ff2ff0f](ff2ff0f)) * use swc pack instead of ncc ([#140](#140)) ([3f9e947](3f9e947)) ### Other chores * add basic e2e test workflow for main action ([#141](#141)) ([414c169](414c169)) * add manual trigger to the tests ([4908a48](4908a48)) * always use lf for build files ([#122](#122)) ([a9b9cfe](a9b9cfe)) * bump semver-regex from 3.1.2 to 3.1.3 ([#114](#114)) ([50c0ef9](50c0ef9)) * bump tar from 4.4.13 to 4.4.19 ([#112](#112)) ([5f5673b](5f5673b)) * bump tmpl from 1.0.4 to 1.0.5 ([#115](#115)) ([92ec93f](92ec93f)) * clean up dependencies and eslint rules ([#145](#145)) ([3ae7999](3ae7999)) * clean up master references in release and workflows ([#104](#104)) ([a2d9b1c](a2d9b1c)) * clean up tsconfig and eslint rules ([#123](#123)) ([c20146f](c20146f)) * create new release 7.0.0 ([d50f5dd](d50f5dd)), closes [#149](#149) [#147](#147) [#121](#121) [#146](#146) [#148](#148) [#139](#139) [#144](#144) [#138](#138) [#137](#137) [#150](#150) [#127](#127) [#126](#126) [#140](#140) [#141](#141) [#122](#122) [#114](#114) [#112](#112) [#115](#115) [#145](#145) [#104](#104) [#123](#123) [#125](#125) [#128](#128) [#142](#142) [#119](#119) [#120](#120) [#116](#116) [#110](#110) [#111](#111) [#107](#107) [#103](#103) * lint root files with prettier and eslint ([#125](#125)) ([1cfa7bb](1cfa7bb)) * rebuild project ([45b79a0](45b79a0)) * rebuild project after dependency bumps ([b099f17](b099f17)) * remove husky and improve ci build message ([#128](#128)) ([39b0d36](39b0d36)) * run test workflow every day at 15:00 utc ([fd33d59](fd33d59)) * simplify review and update build detection ([#142](#142)) ([ee415b0](ee415b0)) * update `setup-node` action to `v2` release ([#119](#119)) ([3b3e6b8](3b3e6b8)) * update semantic release to update the build ([354dc08](354dc08)) * update the action manifest ([633cdf0](633cdf0)) * upgrade dev dependencies and rebuild action ([#120](#120)) ([186ef00](186ef00)) ### Documentation changes * explicitly add non interactive flag ([#151](#151)) ([844ea09](844ea09)) * fix broken sup element in readme ([0b1822a](0b1822a)) * fix github capitalization ([#116](#116)) ([b602469](b602469)) * fix unsplash comment on pr branch reference ([#110](#110)) ([f14253e](f14253e)) * update comment preview readme header ([7a46a49](7a46a49)) * update default values in readme ([c2f11d2](c2f11d2)) * update expo.io links to expo.dev ([#111](#111)) ([1c8e5a2](1c8e5a2)) * update node version reference in readme ([#107](#107)) ([3ba667b](3ba667b)) * update readme and contributing guides ([521de72](521de72)) * update step title in example with eas cli ([7934883](7934883)) * update workflow badge with test status ([99593b8](99593b8)) * use `main` branch to update `v{major}` tag ([#103](#103)) ([def44e2](def44e2))
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Linked issue
Fixes #134
Additional context
Before merging this, I want to add an e2e test for the action(s).