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

Include git SHA in the built bundle #562

Merged
merged 3 commits into from
May 10, 2016
Merged

Include git SHA in the built bundle #562

merged 3 commits into from
May 10, 2016

Conversation

jbeezley
Copy link
Contributor

Fixes #552

@codecov-io
Copy link

codecov-io commented May 2, 2016

Current coverage is 71.74%

Merging #562 into master will increase coverage by +<.01%

@@             master       #562   diff @@
==========================================
  Files            81         82     +1   
  Lines          7069       7070     +1   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           5071       5072     +1   
  Misses         1998       1998          
  Partials          0          0          

Powered by Codecov. Last updated by 6dcb286...9b13120

@aashish24
Copy link
Member

@jbeezley looks good to me. IS CI failing because of some system issue?

@jbeezley
Copy link
Contributor Author

jbeezley commented May 2, 2016

The new test is failing when run from CTest. I'll have to look into it more to find out why.

@jbeezley jbeezley force-pushed the publish-git-sha branch from 08188a2 to 6ddbcd2 Compare May 9, 2016 18:53
@jbeezley jbeezley force-pushed the publish-git-sha branch from adaaa51 to 85fa552 Compare May 9, 2016 19:27
@jbeezley
Copy link
Contributor Author

jbeezley commented May 9, 2016

PTAL. The failure was because execSync wasn't added until node 0.12. I added a warning and updated the dashboard machine.

@manthey
Copy link
Contributor

manthey commented May 10, 2016

LGTM

@jbeezley jbeezley merged commit 1db67be into master May 10, 2016
@jbeezley jbeezley deleted the publish-git-sha branch May 10, 2016 12:19
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.

4 participants