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

[ENG-4579] convert eas-cli to ESM #1199

Closed
wants to merge 11 commits into from
Closed

[ENG-4579] convert eas-cli to ESM #1199

wants to merge 11 commits into from

Conversation

dsokal
Copy link
Contributor

@dsokal dsokal commented Jul 7, 2022

Checklist

  • I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why

Please describe the motivation for this PR, and link to relevant GitHub issues, forums posts, or feature requests.

How

How did you build this feature or fix this bug and why?

Test Plan

Please describe how you tested this change and how a reviewer could reproduce your test, especially if this PR does not include automated tests! If possible, please also provide terminal output and/or screenshots demonstrating your test/reproduction.

@dsokal dsokal changed the title convert eas-cli to ESM [ENG-4579] convert eas-cli to ESM Jul 7, 2022
@linear
Copy link

linear bot commented Jul 7, 2022

ENG-4579 Convert eas-cli to esm

Screenshot 2022-04-07 at 15.26.32.png

We can't convert a bunch of deps in eas-cli because it's not an ESM. It seems that it should be possible to convert eas-cli now

oclif/core#130

https://github.com/typhonjs-oclif-scratch/test-cli-modern/tree/main/src

@github-actions
Copy link

github-actions bot commented Jul 7, 2022

Size Change: +778 kB (+3%)

Total Size: 25.9 MB

Filename Size Change
./packages/eas-cli/dist/eas-linux-x64.tar.gz 25.9 MB +778 kB (+3%)

compressed-size-action

@dsokal
Copy link
Contributor Author

dsokal commented Sep 6, 2022

Closing this for now

@dsokal dsokal closed this Sep 6, 2022
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.

1 participant