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

Framework: Add the commit sha1 information in the environment badge (on hover) #2068

Merged
merged 1 commit into from
Jan 8, 2016

Conversation

Tug
Copy link
Contributor

@Tug Tug commented Jan 4, 2016

This PR fixes a few thing from #1531 :

  • Fixes some code formatting issues
  • Removes a line break at the end of the branch name
  • getCurrentBranchName() now returns a String instead of a Buffer

And adds a new information: the last commit id (or sha1 checksum) available when hovering the branch name in the environment badge

badge-with-hover

Context

This is useful for a multi branch version of the application (See: https://github.com/Automattic/calypso-live-branches). This way we can ensure the server has updated our changes.

@Tug Tug force-pushed the add/badge-commit-id branch 3 times, most recently from 7988ddc to 50503ab Compare January 5, 2016 00:41
@Tug Tug added [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. and removed [Status] In Progress labels Jan 5, 2016
@aduth
Copy link
Contributor

aduth commented Jan 7, 2016

Works well 👍

@Tug Tug force-pushed the add/badge-commit-id branch from 50503ab to 17c575e Compare January 8, 2016 15:47
Tug added a commit that referenced this pull request Jan 8, 2016
Framework: Add the commit sha1 information in the environment badge (on hover)
@Tug Tug merged commit 46a7e93 into master Jan 8, 2016
@Tug Tug deleted the add/badge-commit-id branch January 8, 2016 16:03
@Tug Tug removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Jan 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants