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

Upgrade to Babel 6 #356

Merged
merged 244 commits into from
Jun 7, 2016
Merged

Upgrade to Babel 6 #356

merged 244 commits into from
Jun 7, 2016

Conversation

matthewp
Copy link
Member

This uses the new version of Transpile which uses Babel 6.

matthewp and others added 30 commits November 30, 2015 09:14
Upgrade to Steal 0.13.0
Better minify error messaging
I incorrectly merged the minify module which caused tests to break, this fixes it.
This tests us against Steal 0.14.0-pre.6 which contains progressively loaded package.jsons.
This fixes #365. Some configuration might need to be reused when loading multiple bundles in multiple loaders. With the new progressive loading of package.jsons we need to keep the `npmContext` between loaders because each bundle will have a different set of NPM dependencies, and we need to build them **all** in the final output.

This will also require a fix in system-npm to reuse an existing npmContext.
Reuse some config when loading bundles
Along with #372 this will close #368
This was added when I implemented the Trace extension. Isn't needed, Steal is able to load the browser-based babel and use that for transpiling just fine. Closes #363
Remove dependency on babel
This is the intial streaming API, closing #142

The 4 streams are:

```js
stealTools.createGraphStream()
stealTools.createMultiBuildStream()
stealTools.createConcatStream()
stealTools.createWriteStream()
```

These need to be used in conjuction but allow you to swap in your own streams in the process to do other behaviors (or you could, say, omit the write stream to prevent writing to the file system).

Here's an example usage, this is basically what `stealTools.build` does:

```js
var stealTools = require("steal-tools");

var createGraphStream = stealTools.createGraphStream;
var multiBuild = stealTools.createMultiBuildStream;
var concat = stealTools.createConcatStream;
var write = stealTools.createWriteStream;

var system = {
    config: __dirname + "/package.json!npm"
};

var stream = createGraphStream(system)
    .pipe(multiBuild())
    .pipe(concat())
    .pipe(write());
```

This PR includes docs and a guide that explains the usage better, see `doc/guides/streams.md`.
When doing a multi-main build make_graph_with_bundles was mutating the `main` which caused the first main to be excluded from the build.

This fixes #375
Add test to verify s.graph works with es6 modules our transpiler doesn't support
Correctly normalize when ignore: false is used
This updates transform's `exports` property so that it can be used to specify values that should be exported.

For example if you have a module **foo** that looks like:

```js
module.exports = "foo bar";
```

And you set exports:

```js
exports: {
  "foo": "foo.bar"
}
```

Then it will set it so that `window.foo.bar === "foo bar"`.
This adds the `+standalone` helper and makes sure it works with the new `exports` option.
Use exports to specify global exported values
This documents export.ignore better, breaking out a typedef for the **ignorerer** function that is optional. Closes #437
Add documentation on export.ignore
Updates to the progressively loaded guide
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling ce27cd1 on babel6 into * on major*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling c2e4a8c on babel6 into * on major*.

@coveralls
Copy link

coveralls commented Jun 7, 2016

Coverage Status

Changes Unknown when pulling e6409e1 on babel6 into * on major*.

Removing 0.10, will be replaced by 6 in #455
@matthewp matthewp merged commit 846bf4c into major Jun 7, 2016
@matthewp matthewp deleted the babel6 branch June 7, 2016 14:30
@coveralls
Copy link

coveralls commented Jun 7, 2016

Coverage Status

Changes Unknown when pulling 089b866 on babel6 into * on major*.

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.

None yet

5 participants