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

adding updated logo and testing out travis build #5

Merged
merged 1 commit into from
Mar 21, 2020
Merged

Conversation

vsoch
Copy link
Collaborator

@vsoch vsoch commented Mar 21, 2020

This quick PR will (possibly unwisely!) try to do two quick changes to:

  • update the logo from Add a logo for the python package?  #4
  • update the travis.yml to only build master (and possibly this will work with the backend settings on travis so the PR will build still?

What we'll need to do after merge is update the README.md to have absolute links to the files, that way they will render cleanly in the next pypi release (relative paths do not).

Signed-off-by: vsoch vsochat@stanford.edu

Signed-off-by: vsoch <vsochat@stanford.edu>
@codecov
Copy link

codecov bot commented Mar 21, 2020

Codecov Report

Merging #5 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   72.56%   72.56%           
=======================================
  Files          15       15           
  Lines         452      452           
=======================================
  Hits          328      328           
  Misses        124      124           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 93d7fcb...a0146aa. Read the comment docs.

@vsoch
Copy link
Collaborator Author

vsoch commented Mar 21, 2020

Let's wait on discussion in #4 to make sure the colors are consistent (I can update the files here!) And either way, looks like the Travis issue was an easy fix!

@vsoch
Copy link
Collaborator Author

vsoch commented Mar 21, 2020

okay, colors will do for now. Merging!

@vsoch vsoch merged commit 97a4c3f into master Mar 21, 2020
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.

2 participants