Dependency, linting, and compression updates #437
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
* Update angular2 to latest beta * Update typings to latest * Compress JS and CSS * Update CSS linting config * Remove some CSS lint violations * Enable prod mode when `config["environment"] === "production"` * Move /app/assets to /assets and make them work in the dist
|
By analyzing the blame information on this pull request, we identified @ryankeairns, @fnichol and @jtimberman to be potential reviewers |
|
|
smith
pushed a commit
that referenced
this pull request
Apr 25, 2016
Make it so when we `npm run dist`, it takes a SHA256 of app.js and app.css and uses those filenames in the built code. This means we can cache these files forever and users won't have to bust the cache. Also: * Fix some CSS and JS linting issues * Split the long npm scripts into separate script files * Shrinkwrap the stuff that was added in #437
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.
config["environment"] === "production"