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

Clarify schedule is guideline #272

Merged
merged 4 commits into from
Oct 9, 2023

Conversation

Carreau
Copy link
Contributor

@Carreau Carreau commented Oct 2, 2023

I think otherwise the spec page seem a bit harsh.

spec-0000/index.md Outdated Show resolved Hide resolved
@Carreau
Copy link
Contributor Author

Carreau commented Oct 2, 2023 via email

spec-0000/index.md Outdated Show resolved Hide resolved
spec-0000/index.md Outdated Show resolved Hide resolved
@munkm
Copy link
Member

munkm commented Oct 2, 2023

A more broad comment on the wording we're converging on here: is it really the technicalities of when to drop that is confusing? In my experience the big issue is what people think of when "drop" is brought up. Maybe we want to add a disclaimer that says something to the effect of:

Disclaimer on this SPEC: Drop here is not meant to convey removing support for specific versions of upstream packages. Instead, we intend to ease maintenance burden by providing a schedule where upstream version support is not guaranteed to be supported. This way maintainers have a specific window of support they may commit to without adding maintenance burden of deprecations, etc.

spec-0000/index.md Outdated Show resolved Hide resolved
Copy link
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

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

Overall I like the softening as this drop schedule is often misquoted and used to somewhat aggressively demand a drop (which is quite annoying for developer infrastructure or when a lot of the optional dependencies are not yet compatible with the new version).

@Carreau
Copy link
Contributor Author

Carreau commented Oct 2, 2023 via email

@bsipocz
Copy link
Member

bsipocz commented Oct 2, 2023

I agree with @Carreau here. While some parts (maybe even 99.9%) keep working when dropping support, removing from CI but not explicitly changing python_requires tended to lead to more problems and a lot of headaches of sorting out bugreports.

I also like to be very explicit with minimum versions for dependencies, even if an older one might work, I bake in the minimum that I test with and don't rely on wildcarded versions either.

Co-authored-by: Brigitta Sipőcz <[email protected]>
spec-0000/index.md Outdated Show resolved Hide resolved
@munkm
Copy link
Member

munkm commented Oct 2, 2023

While some parts (maybe even 99.9%) keep working when dropping support, removing from CI but not explicitly changing python_requires tended to lead to more problems and a lot of headaches of sorting out bugreports.

That's definitely not what I was trying to imply that we suggest here. I was trying to make it clear that this SPEC isn't intended to add maintenance burden for lower support versions (in both ways -- by not keeping some lower version supported for all time, but also not that dropping support needs to be some huge deal every quarter)? That was my impression from our conversations. Is there a way to be clearer about this so it isn't interpreted aggressively like we've seen in some cases?

@bsipocz
Copy link
Member

bsipocz commented Oct 2, 2023

Ahh, ok, sorry, then we're all on the very same page :)

@jarrodmillman jarrodmillman merged commit cc0df45 into scientific-python:main Oct 9, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants