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

Changed the homepage link to cinematography #708

Merged
merged 4 commits into from
Dec 9, 2016
Merged

Conversation

jamieu
Copy link
Member

@jamieu jamieu commented Dec 7, 2016

Small change to cinematography link

@codecov-io
Copy link

codecov-io commented Dec 9, 2016

Current coverage is 99.52% (diff: 100%)

Merging #708 into master will decrease coverage by 0.20%

@@             master       #708   diff @@
==========================================
  Files           101        101          
  Lines          1880       1901    +21   
  Methods         178        186     +8   
  Messages          0          0          
  Branches        474        490    +16   
==========================================
+ Hits           1875       1892    +17   
- Misses            5          9     +4   
  Partials          0          0          

Powered by Codecov. Last update 42119a7...bac2991

Copy link
Contributor

@SimonLab SimonLab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @jamieu 👍

@Danwhy Danwhy removed their assignment Dec 9, 2016
@nelsonic
Copy link
Contributor

nelsonic commented Dec 9, 2016

merging. 👍

@nelsonic nelsonic merged commit 3b952ad into master Dec 9, 2016
@nelsonic nelsonic deleted the homepage-links branch December 9, 2016 15:51
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.

5 participants