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 support for detecting previous stable version #50

Merged
merged 1 commit into from
Jun 7, 2018

Conversation

jatcwang
Copy link
Contributor

@jatcwang jatcwang commented Dec 4, 2017

This command always return the last tagged version before the current HEAD.
Most of the work was done by @MaxwellBo. I just fixed some issues found in usage and cleaned it up slightly.

  • Update SBT cross build version to 1.0.4 and update configs according to SBT crossbuilding guide

NOTE

When I run ^ mimaReportBinaryIssues it seems to only run the task for the current SBT version. Manually switching SBT cross build version using ^^ 1.0.4 then mimaReportBinaryIssues works fine. Maybe you can spot what I'm doing wrong but this is what I've done for other SBT plugins and it seems to work fine there

@dwijnand
Copy link
Member

dwijnand commented Jun 1, 2018

Sorry for never giving any feedback on this.

Where does this PR stand currently? Are you still interested in this change? Given the merge conflicts, have the changes in master impacted this PR?

I've been feeling the mild itch that this feature should really exist in sbt-dynver, so I'm interested in trying to land it.

Alternatively my OCD is eager to close this PR if it's abandoned. 😄

@jatcwang
Copy link
Contributor Author

jatcwang commented Jun 1, 2018

The conflicts seems minor. I'll fix this up

@dwijnand
Copy link
Member

dwijnand commented Jun 1, 2018

Thank you!

@jatcwang
Copy link
Contributor Author

jatcwang commented Jun 3, 2018

I'm working on more test cases and will also add documentation

@jatcwang
Copy link
Contributor Author

jatcwang commented Jun 5, 2018

This is ready for review :)

This command returns the last stable tagged version before the current HEAD. If a tag cannot be found for any reason (no parent commit or no tag) then the result is None.
Copy link
Member

@dwijnand dwijnand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is wonderful. Thank you @jatcwang and @MaxwellBo!

@dwijnand dwijnand merged commit 1be6492 into sbt:master Jun 7, 2018
@2m
Copy link
Member

2m commented Jun 8, 2018

This is awesome. Thank you @jatcwang and @MaxwellBo!

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.

3 participants