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

Publish 0.7.0 #59

Closed
mdedetrich opened this issue Feb 12, 2024 · 13 comments
Closed

Publish 0.7.0 #59

mdedetrich opened this issue Feb 12, 2024 · 13 comments
Assignees

Comments

@mdedetrich
Copy link
Contributor

So we can align on when to publish so we don't miss anything

@mkurz
Copy link
Member

mkurz commented Feb 13, 2024

Almost done.
I pushed a milestone release, to check if publishing works: https://github.com/sbt/sbt-paradox-material-theme/releases/tag/v0.7.0-M1
Also before final release we want to merge:

And maybe it makes sense to take a look at

which also may be a fix for

@mdedetrich
Copy link
Contributor Author

I have been thinking, maybe the 0.7.0/1.0.0 distinction between sbt-paradox/sbt-paradox-theme is not necessary right now and we should just release as 0.7.0 with sbt-paradox/sbt-paradox-theme versions pinned to 0.9.x and only bump sbt-paradox/sbt-paradox-theme to 1.0.0 when we find an actual binary incompatibility between sbt-paradox/sbt-paradox-theme 0.9.x and 0.11.x+.

The sbt-paradox/sbt-paradox-theme API that this project uses is stable from 0.9.x all the way until the latest versions, the actual reason behind the incompatibility are dependencies of sbt-paradox (iirc scala-xml/parboiiled) that are split between supporting JDK 1.8 and JDK 11+ but we never use those dependencies directly.

Thats why doing this trick works.

@mkurz wdyt?

@mkurz
Copy link
Member

mkurz commented Feb 14, 2024

and only bump sbt-paradox/sbt-paradox-theme

We can not bump these two libraries to 1.x ourselves anyway, because they are hosted under the lightbend org and I do not have access to that repos. I guess you mean this repo here (sbt-paradox-material-theme).

So do I get this right, let's release 0.7.0 with paradox deps pinned to 0.9.2 (actually that is what we wanted to do anyway), branch off a 0.7.x branch from that tag, and then on main upgraded the paradox deps to 0.10.6 and release 0.8.0 (instead of 1.0.0)?

@mkurz
Copy link
Member

mkurz commented Feb 14, 2024

@mdedetrich Everything done, all PRs merged, also M1 got published without problems.

You want to tag/push 0.7.0?

@mdedetrich
Copy link
Contributor Author

So do I get this right, let's release 0.7.0 with paradox deps pinned to 0.9.2 (actually that is what we wanted to do anyway), branch off a 0.7.x branch from that tag, and then on main upgraded the paradox deps to 0.10.6 and release 0.8.0 (instead of 1.0.0)?

Well actually I was thinking of just continuing to work on 0.7.x unless we are forced to upgrade paradox deps to 0.10.6 (or later by the time we get to it) because there isn't actually any reason to upgrade the paradox deps and doing so can cause annoyances with JDK 1.8.

@mkurz
Copy link
Member

mkurz commented Feb 14, 2024

Hmmm... Not sure... I think you see this too much from the perspective of Pekko.
There are other projects that use this library so I am not sure it would be "fair" to hold dependency upgrades back "just" because of Pekko.
What I would do now is: releae 0.7.0 (which we do anyway), branch off 0.7.x so you can still cut 0.7.1 etc. if needed, but on master merge #66 and cut 0.8.0.
To be honest I do not expect great changes to this library currently so it's just about also having a release with all the deps up to date for other projects.

@mdedetrich
Copy link
Contributor Author

mdedetrich commented Feb 14, 2024

There are other projects that use this library so I am not sure it would be "fair" to hold dependency upgrades back "just" because of Pekko.

Unless I am missing something its not going to impact any project either way, as we speak now you can easily load sbt-paradox-material-theme with sbt-paradox/sbt-paradox-theme 0.10.6 without any issues (I can even write an sbt-scripted test to prove this).

Its the reverse i.e. downgrading the sbt-paradox/sbt-paradox-theme as is here due to JDK 8 that is problematic and thats what Pekko is dealing with.

To be honest I do not expect great changes to this library currently so it's just about also having a release with all the deps up to date for other projects.

There are going to be big changes, see apache/pekko#1097 (comment) . For example the material theme is seriously outdated and an upcoming version (9.x) is meant to be provide much better improved search functionality and its likely to be a big change.

@mdedetrich
Copy link
Contributor Author

mdedetrich commented Feb 14, 2024

See #67 as a demonstration that this project currently works completely fine with latest version of sbt-paradox/sbt-paradox-theme

@mkurz
Copy link
Member

mkurz commented Feb 14, 2024

Ok, then we just tag 0.7.0 and we are done I guess?

@mdedetrich
Copy link
Contributor Author

Ok, then we just tag 0.7.0 and we are done I guess?

Yup!

@mkurz
Copy link
Member

mkurz commented Feb 14, 2024

Please do it, I am out of office on my phone right now

@mdedetrich
Copy link
Contributor Author

Please do it, I am out of office on my phone right now

Tag pushed and action triggered https://github.com/sbt/sbt-paradox-material-theme/actions/runs/7901103220 . Going to bed now, will check that its on maven tomorrow and then make release notes (if you haven't done so by the time I wake up).

@mdedetrich
Copy link
Contributor Author

Now available on maven (see apache/pekko-sbt-paradox#92), closing issue as completed and will make release notes

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

No branches or pull requests

2 participants