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

Support Mill 0.11.0-M1 #36

Merged
merged 1 commit into from
Dec 22, 2022
Merged

Support Mill 0.11.0-M1 #36

merged 1 commit into from
Dec 22, 2022

Conversation

lolgab
Copy link
Contributor

@lolgab lolgab commented Dec 21, 2022

I used a snapshot release of de.tobiasroeser.mill.vcs.version, you may want to wait for the next stable.

@lolgab lolgab marked this pull request as ready for review December 21, 2022 18:12
@ckipp01
Copy link
Owner

ckipp01 commented Dec 21, 2022

I used a snapshot release of de.tobiasroeser.mill.vcs.version, you may want to wait for the next stable.

Hey! Thanks for the pr! Ideally I'd like to include a stable release if I was going to release, but I'm also comfortable merging this, which will produce a snapshot that you could use. Would that work for you? As I'm assuming you're wanting to use this with something that is using 0.11.0-M1 😆

@lolgab
Copy link
Contributor Author

lolgab commented Dec 21, 2022

I'm not waiting on it, so no worries :)
We can expect @lefou to publish a stable version soon :)
I went through all my plugins to prepare them for 0.11
Also I want to use the performance improvements I added to zinc in my own builds as soon as possible so I created this PR :)

@lefou
Copy link

lefou commented Dec 22, 2022

We can expect @lefou to publish a stable version soon :)

Not so sure about that. We have a not so small backlog and we should try to bring as much as possible into 0.11, to avoid a shortly following yet incompatible 0.12. Beside some speed improvements in 0.11, Mill 0.10 is pretty much recent.

I try to publish a new 0.11 milestone whenever the API changes, so that whoever needs to stay on the bleeding edge, has at least an option to target the binary platform.

@lolgab
Copy link
Contributor Author

lolgab commented Dec 22, 2022

We can expect @lefou to publish a stable version soon :)

Not so sure about that. We have a not so small backlog and we should try to bring as much as possible into 0.11, to avoid a shortly following yet incompatible 0.12. Beside some speed improvements in 0.11, Mill 0.10 is pretty much recent.

I try to publish a new 0.11 milestone whenever the API changes, so that whoever needs to stay on the bleeding edge, has at least an option to target the binary platform.

I was referring to a stable version of mill.vcs.version for Mill 0.11.0-M1, not a stable version of Mill :)

@lefou
Copy link

lefou commented Dec 22, 2022

I was referring to a stable version of mill.vcs.version for Mill 0.11.0-M1, not a stable version of Mill :)

Oh. 😳 I already published a snapshot supporting Mill 0.11.0-M1: https://repo1.maven.org/maven2/de/tototec/de.tobiasroeser.mill.vcs.version_mill0.11.0-M1_2.13/0.3.0-7-8f12c7/

That's the one already used in this PR. I didn't plan to make a numbered release of it, as it is already accessible from Maven Central. Do you need one?

@lolgab
Copy link
Contributor Author

lolgab commented Dec 22, 2022

If we think it's ok for this plugin to depend on a snapshot release of it, it's fine for me :)

@ckipp01
Copy link
Owner

ckipp01 commented Dec 22, 2022

I guess I have opinionated thoughts about snapshots. Mainly if we're ok releasing and having others depend on them, why not just make it a stable release? I can merge this, but my snapshot will go into sonatype snapshots, which means you'd need to add another resolver and doing that for plugins is a bit of a hassles since you have to add a @ up top.

I typically like to avoid relying on SNAPSHOTs for stable releases, but I see there is hardly anything that's changed in the commits up until this, and I know @lefou takes compatibility seriously 😆 .

@ckipp01 ckipp01 merged commit 5d4b71e into ckipp01:main Dec 22, 2022
@lolgab lolgab deleted the mill-0.11 branch December 22, 2022 15:14
@lefou
Copy link

lefou commented Dec 22, 2022

So, technically, mill-vcs-version-0.3.0-7-8f12c7 is a proper release. It's on Maven Central releases repository, and as such rather immutable. The version is also unique. The only thing that's different from other "official" releases is, that it is not tagged with a git tag and it has no dedicated changelog entry. But it already has the (start of the) commit hash in it's name, so it should be still reproducible.

So it depends where @ckipp01 cut the line between "snapshots" and "releases". Is it semantically (Maintainer created a tag) or technically (the (probably not unique) location of the binary artifact)?

Because of no other changes beside some library dependency bumps, I find a new release kind of overkill. If it would be technically easier, I would in fact try to release the already released version 0.3.0 against the new Mill API, but it's simply not worth the efforts to do it, given the fact, that there will be more of these -Mx Mill releases.

@ckipp01
Copy link
Owner

ckipp01 commented Dec 22, 2022

No worries at all. I just released 0.1.5 @lolgab, so you should be able to use that.

Thinking about it, maybe both of us should have used an RC release ha.

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.

None yet

3 participants