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

artifact-migrations.conf: slick-migration-api[-flyway] #2018

Merged
merged 2 commits into from
Mar 20, 2021

Conversation

nafg
Copy link
Contributor

@nafg nafg commented Mar 19, 2021

No description provided.

@nafg
Copy link
Contributor Author

nafg commented Mar 19, 2021

It wasn't clear if initialVersion should be the last version before the change or the first version after the change. I did the former.

@codecov
Copy link

codecov bot commented Mar 19, 2021

Codecov Report

Merging #2018 (40ecf54) into master (f07183f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2018   +/-   ##
=======================================
  Coverage   78.58%   78.58%           
=======================================
  Files         131      131           
  Lines        2251     2251           
  Branches       54       54           
=======================================
  Hits         1769     1769           
  Misses        482      482           

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 f07183f...40ecf54. Read the comment docs.

@fthomas
Copy link
Member

fthomas commented Mar 19, 2021

the first version after the change

initialVersion should be that.

@nafg
Copy link
Contributor Author

nafg commented Mar 19, 2021

Ok updated. You can squash merge if you like

@fthomas fthomas added this to the 0.10.0 milestone Mar 20, 2021
@fthomas fthomas merged commit 21c5160 into scala-steward-org:master Mar 20, 2021
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