⚡ sourcemaps, a better npm run release, and an updated README.md
#147
Conversation
This file contains hidden or 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
Rather than having to set an environment variable to say you don’t have the cert, let the build tooling figure it out for you and log more useful messages along the way. If you have the certificate, you’ll now see:  And if you don’t, you’ll no longer see a really confusing stack trace but instead: 
Is it documentation? Is it a build dashboard? You be the judge.
|
LGTM |
kangas
added a commit
that referenced
this pull request
Oct 5, 2015
⚡ sourcemaps, a better `npm run release`, and an updated README.md
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
README.mdis our front doorImproved the setup instructions, updated and added to our summary of key modules that make up this project, and provided a better context for
npm run release. Even better IMO, all of our modules have tests and run in CI... So if you got it, flaunt it.sourcemaps
This will make us hugely more productive. When debugging the app or when we receive a stack trace from bugsnag, we'll actually get the real filename and line number instead of
index.js:84980:13 · Socket._onerror.auto detect if you can sign builds
We've had a lot of new folks checking out the main compass repo and looking at how the sausage is made. Unfortunately, the first thing they run into has been an error which is a death nail to them becoming active contributors, which we could so desperately benefit from.
So now, rather than having to set an environment variable to say you don’t have
the cert, let the build tooling figure it out for you and log more useful messages along the way.
If you have the certificate, you’ll now see:
And if you don’t, you’ll no longer see a really confusing stack trace but instead: