Skip to content

Make babel-standalone an ESModule and enable flow#9025

Merged
danez merged 3 commits intomasterfrom
standalone-es
Mar 6, 2019
Merged

Make babel-standalone an ESModule and enable flow#9025
danez merged 3 commits intomasterfrom
standalone-es

Conversation

@danez
Copy link
Member

@danez danez commented Nov 15, 2018

Q                       A
Fixed Issues?
Patch: Bug Fix? n
Major: Breaking Change? n
Minor: New Feature? n
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

As a requirement for #7581, but it is also not abad thing to do for webpack so treeshaking and module concatenation can work.
There also should not be any problems with circle dependencies as standalone is the entry point for webpack and is nowhere referenced.

@danez danez added PR: Polish 💅 A type of pull request used for our changelog categories pkg: standalone labels Nov 15, 2018
@babel-bot
Copy link
Collaborator

babel-bot commented Nov 15, 2018

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/10391/

@@ -0,0 +1,244 @@
// @flow
export { default as externalHelpers } from "@babel/plugin-external-helpers";
Copy link
Member

Choose a reason for hiding this comment

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

Is it possible to auto-generate this file and the list in packages/babel-standalone/src/index.js?

Copy link
Member Author

Choose a reason for hiding this comment

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

I will do that.

Copy link
Member

Choose a reason for hiding this comment

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

Note that we shouldn't generate it from the list of the directories but we still need to maintain a list of included plugins (which wouldn't be duplicated anymore).

@danez danez added the PR: WIP label Dec 3, 2018
@jasonLaster
Copy link
Contributor

Looks good. I like the flow types. Out of curiosity could those land separately?

@danez
Copy link
Member Author

danez commented Feb 20, 2019

Okay I finally added the autogeneration.

@@ -0,0 +1,87 @@
{
"all": [
Copy link
Member

Choose a reason for hiding this comment

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

Can't this file just be the array?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh yes. I first had more keys in there, as I wanted to also specify which plugins go to which preset, but that does not work because of plugin options and other business logic.
Will revert.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@nicolo-ribaudo
Copy link
Member

The failing status of this PR is just because the hack I used to create the publish github action without support for "release" events in publis repos has some problems when PRs are opened from branches of this repo.

@danez danez merged commit fb81e8f into master Mar 6, 2019
@danez danez deleted the standalone-es branch March 6, 2019 22:30
mAAdhaTTah added a commit to mAAdhaTTah/babel that referenced this pull request Mar 15, 2019
* master: (58 commits)
  Remove dependency on home-or-tmp package (babel#9678)
  [proposal-object-rest-spread] fix templateLiteral in extractNormalizedKeys (babel#9628)
  Partial application plugin (babel#9474)
  Private Static Class Methods (Stage 3) (babel#9446)
  gulp-uglify@3.0.2
  rollup@1.6.0
  eslint@5.15.1
  jest@24.5.0
  regexpu-core@4.5.4
  Remove input and length from state (babel#9646)
  Switch from rollup-stream to rollup and update deps (babel#9640)
  System modules - Hoist classes like other variables (babel#9639)
  fix: Don't transpile ES2018 symbol properties (babel#9650)
  Add WarningsToErrorsPlugin to webpack to avoid missing build problems on CI (babel#9647)
  Update regexpu-core dependency (babel#9642)
  Add placeholders support to @babel/types and @babel/generator (babel#9542)
  Generate plugins file
  Make babel-standalone an ESModule and enable flow (babel#9025)
  Reorganize token types and use a map for them (babel#9645)
  [TS] Allow context type annotation on getters/setters (babel#9641)
  ...
@nicolo-ribaudo nicolo-ribaudo mentioned this pull request Mar 21, 2019
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 4, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue pkg: standalone PR: Polish 💅 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants