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

Fix README.md and build issues #7

Merged
merged 4 commits into from
Sep 28, 2018
Merged

Fix README.md and build issues #7

merged 4 commits into from
Sep 28, 2018

Conversation

jeffmay
Copy link
Contributor

@jeffmay jeffmay commented Sep 26, 2018

@rallyhealth/engineers @mingy711 @sklei2

So I started by trying to fix the badges in the README.md, but CI pointed out that my build issue from before is still an issue.

In the process of fixing these issues, I went down a rabbit hole chasing this bug lightbend-labs/mima#206

  • Fix the badges in the README.md to show the latest versions, code coverage, and build status
  • Upgrade to sbt 1.0
  • Upgrade to Play 2.6.19
  • Upgrade sbt-git-versioning plugin

@jeffmay jeffmay force-pushed the fix-badges branch 2 times, most recently from 108bbc2 to 4f184c2 Compare September 28, 2018 02:25
@jeffmay jeffmay changed the title Fix README.md badges and use markdown table Fix README.md and build issues Sep 28, 2018
@codecov-io
Copy link

codecov-io commented Sep 28, 2018

Codecov Report

Merging #7 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #7   +/-   ##
=======================================
  Coverage   91.11%   91.11%           
=======================================
  Files           5        5           
  Lines          90       90           
  Branches       10       10           
=======================================
  Hits           82       82           
  Misses          8        8

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 85f79ad...199340f. Read the comment docs.

- Upgrade to sbt 1.0 to solve a weird bug with MiMa
  (lightbend-labs/mima#206)
- Add missing resolver
- Remove name supression config
- Fix resolver issue
- +test will download both versions of scala, so we were testing
  both versions twice and slowing down the build
@jeffmay jeffmay merged commit 5591a39 into master Sep 28, 2018
@jeffmay jeffmay deleted the fix-badges branch September 28, 2018 22:14
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