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

Improvements to the cli #212

Merged
merged 2 commits into from Apr 6, 2015
Merged

Improvements to the cli #212

merged 2 commits into from Apr 6, 2015

Conversation

matthewp
Copy link
Member

@matthewp matthewp commented Apr 6, 2015

This adds a bunch of improvements to the cli like aliases for --config and --main, makes build the default command, package.json!npm the default config. Also adds some tests, which we didn't have before. Closes stealjs/steal#352

Changes

  1. The flags --config and --main have been aliased to -c and -m respectively.
  2. build is now the default command, meaning you don't have to specify steal-tools build every time.
  3. By default the config is package.json!npm. This means if you're using the npm plugin you only need to type steal-tools to get a build going.
  4. Options added for --source-maps and --source-maps-content to toggle on source maps behavior. See the documentation for what these mean.
  5. Option --no-minify added to disable minification.

See all options available with steal-tools --help.

Examples

A normal build can now be:

steal-tools -c config.js -m app/app

Or if using the npm plugin it is just:

steal-tools

To create source maps add the flag:

steal-tools --source-maps

This adds a bunch of improvements to the cli like aliases for `--config` and `--main`, makes `build` the default command, `package.json!npm` the default config.  Also adds some tests, which we didn't have before. Closes stealjs/steal#352
matthewp added a commit that referenced this pull request Apr 6, 2015
@matthewp matthewp merged commit 5572f65 into minor Apr 6, 2015
@matthewp matthewp deleted the better-cli branch April 6, 2015 18:17
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

1 participant