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

Add dependabot and last commit date badges #237

Merged
merged 1 commit into from Mar 19, 2020

Conversation

CasperWA
Copy link
Member

@CasperWA CasperWA commented Mar 19, 2020

Also, use svg images for the heroku badge.

See the resulting README here.

I have spent some time looking into other heroku badge services, in order to see if we can get a logo on there as well. Unfortunately, there doesn't seem to be a straightforward way of doing this. The service we are currently using are showing us images from its repo, no redirecting through shields.io or something similar. Another service here do use badges generated from shields.io, including logo, but we would need to deploy it on our heroku account. In the end, it seems like too much work for a logo - I'd just stay with what we have now, since it works fine.
Just a little comment for the future :)

Also, use svg images for the heroku badge.
@CasperWA CasperWA added enhancement New feature or request priority/low Issue or PR with a consensus of low priority labels Mar 19, 2020
@CasperWA CasperWA requested review from ml-evs and shyamd March 19, 2020 11:04
@codecov
Copy link

codecov bot commented Mar 19, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #237   +/-   ##
=======================================
  Coverage   87.33%   87.33%           
=======================================
  Files          43       43           
  Lines        1943     1943           
=======================================
  Hits         1697     1697           
  Misses        246      246           
Flag Coverage Δ
#unittests 87.33% <ø> (ø)

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 098aa32...f0a299a. Read the comment docs.

@shyamd shyamd merged commit 55c84c3 into Materials-Consortia:master Mar 19, 2020
@shyamd
Copy link
Contributor

shyamd commented Mar 19, 2020

Looks good. Thanks!

@CasperWA CasperWA deleted the add_dependabot_badge branch March 19, 2020 14:42
@ml-evs ml-evs added this to In progress in Road to optimade-python-tools 1.0 via automation Mar 31, 2020
@ml-evs ml-evs moved this from In progress to Done in Road to optimade-python-tools 1.0 Mar 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority/low Issue or PR with a consensus of low priority
Development

Successfully merging this pull request may close these issues.

None yet

2 participants