I'm on release duty today. It looks like 2.11.x ha...
# development
e
I'm on release duty today. It looks like 2.11.x has 1 pending fix that Benjy is on top of in addition to a Pex upgrade I'd like to get in that fixes a Pex lock file issue. Let me know, or add to the milestone if you have others that should block release. I'll probably start that this afternoon assuming Benjy's fix is in and cherry-picked. I'll probably spin up the 2.12.0.dev2 release at that same time unless anyone yells.
h
Yes, thanks. I can try this morning to remove some of the deprecation is that we keep punting on for dev2.
1
e
Ok, great. I had 4 dep PRs on my plate when I looked at it this morning after a glance. I'll assume you're doing those then instead until I hear otherwise in which case I plan on doing them.
👍 1
FWIW, just like labels for PRs are a bad idea IMO (it's good to review what went on in the past X period of time when releasing to spread the knowledge), I think similar about deprecation removals.
h
The subsystem API deprecation must be punted on again, as tool chain has not yet released its new plug-in. I think the others can be handled
e
That's an interesting case. For anyone except TC we would just be breaking them (unknowingly but with fair warning by Pants own terms). I think this is a smell. It seems to say our deprecation policy and aggressiveness is too hard for real users.
h
true, @bitter-ability-32190 it was a big deprecation to rewrite your plug-in to use a whole new API. What do you think about delaying until 13?
b
I'm not sure I agree about us "just breaking them" I expect almost all plugins are in-repo, which means people can fix when they upgrade Pants versions. TC is the odd duck who publishes a plugin. That said, yeah punting costs us nothing 🙂
👍 1
h
@witty-crayon-22786 the changes in target.py a bit complex and I suspect you already had ideas for what the new interface should look like. Would you have a couple minutes to please make those changes?
huh scalac-plugins is also quite involved. @ancient-vegetable-10556 is it simply deleting all of the code related to global plugins? Would you have a couple of minutes to do the removal?
e
I expect almost all plugins are in-repo
Yeah, that's almost certainly true today. That does not seem like the road to a vibrant ecosystem long term though.
c
Yeah, perfect time now to tweak the process, if needed (our plugin at work are not in-repo btw)
👍 2
e
At some point we need an API folks can rely on to get that ecosystem. We need not be Linux and never break user space. But we could be a heck of a lot closer to that end of the spectrum than we are. Today we aren't forced into the general thorny issue of API design that imposes on you.
Guava did this well. Maybe still does. Strange - since Google is generally really bad about this sort of thing!
w
i’ve been tracking (what i think are) issues which block trying to make a push for plugin API stability: https://github.com/pantsbuild/pants/projects/19 would be interested in people’s thoughts on the list, but until we’ve fixed some of the fundamental stuff (particularly around the rule graph), i think we’d be hamstringing ourselves to declare stability.
👍 1
@enough-analyst-54434: regarding the issue that Benjy is on: i don’t think that this is the last RC for 2.11.x, so i wouldn’t wait unless it looks like a PR is headed toward green. i think that we should expect more issues reported once we get the current round of resolve fixes out
h
He already has the cherry pick up for the fix
🎉 1
e
I have 0 opinions on the matter; so I'll hold back on 2.11.x unless I get a clear positive go, go, go!
b
would be interested in people’s thoughts on the list
Might be worth a different 🧵
w
yeeep. sorry.
1
h
My cherrypick is in on 2.11.x, thanks
e
Sounds good. Per Stu above I'll be not doing a 2.11.x today as it turns out.
Or maybe that's not what he meant and he meant cut sooner. At any rate, since I'm confused I'll do no harm until I hear definitely cut 2.11.x-rc-next.
w
@enough-analyst-54434: i was suggesting releasing without it, so the fact that it has landed is icing. sorry that that was confusing
so yes: “go, go, go!”
e
Gotcha. Probably confusing because I didn't read well and I'm also waiting on myself with the Pex upgrade & CP. I will cut a 2.11.x rc then once my self wait terminates.
Folks, I'm going to push of the 2.12 dev release until I can suss https://pantsbuild.slack.com/archives/C0D7TNJHL/p1649463452860439 since (stable) releases are broken and - IIUC - no user is hanging on a 2.12 dev release. Correct me if you know otherwise though.
h
I think that 2.12 should be fixed already https://github.com/pantsbuild/pants/pull/14922. We only did not cherry pick to 2.11. Sorry that I did not finish with the other deprecation beyond the lock file generator. The presentation today took more time than expected. Feel free to bump those deprecation if you would like, they all seem pretty involved