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

Use sbt (1.0's!) dependencyResolution #551

Merged
merged 1 commit into from
Sep 16, 2020

Conversation

dwijnand
Copy link
Collaborator

... instead of reinventing a variant of it.

Fixes #550

@dwijnand
Copy link
Collaborator Author

Review by @alexarchambault, if possible!

... instead of reinventing a variant of it.
@dwijnand dwijnand force-pushed the just-dependencyResolution branch from e04a202 to 21f5f0e Compare September 15, 2020 08:33
@dwijnand dwijnand merged commit d95a6c0 into lightbend-labs:master Sep 16, 2020
@dwijnand dwijnand deleted the just-dependencyResolution branch September 16, 2020 11:14
@alexarchambault
Copy link
Contributor

Sorry for the late review. I would have to check, but I think, the default dependencyResolution may not pull old versions of modules available in the current build, which may not be what we want here.

@dwijnand
Copy link
Collaborator Author

Ah, inter-project resolver things "fixing" the version?

@dwijnand
Copy link
Collaborator Author

I think https://github.com/lightbend/mima/blob/master/sbtplugin/src/sbt-test/sbt-mima-plugin/minimal/test is proof it works (especially the last part "remove all filters so mima check fails")

@rdesgroppes
Copy link

Thanks, guys!

@raboof raboof added this to the 0.8.1 milestone Oct 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

sbt-mima-plugin 0.8.0: why not use project's dependencyResolution by default?
4 participants