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

deprecated-3.7: remove it in master? #2038

Closed
danstowell opened this issue May 7, 2016 · 5 comments
Closed

deprecated-3.7: remove it in master? #2038

danstowell opened this issue May 7, 2016 · 5 comments
Labels
comp: class library SC class library
Milestone

Comments

@danstowell
Copy link
Member

There are some old methods in the deprecated/3.7 folder. Expected practice would be to remove them entirely from 3.8, i.e. from master. Do it?

@danstowell danstowell added the comp: class library SC class library label May 7, 2016
@danstowell danstowell added this to the 3.8 milestone May 7, 2016
@bagong
Copy link
Contributor

bagong commented May 8, 2016

Yep. The Document related stuff keeps bugging headless users ;)

@crucialfelix
Copy link
Member

crucialfelix commented May 8, 2016

You could also add a deprecated/3.8

We do now have an issue if we are doing much faster releases where methods will drop away too quickly for the casual user (ie. the vast majority of normal sane people).

If we are doing a June 1 release that is only 3 months. Maybe we should move the release back to August. Let's discuss what a good regular release period is.

We could agree to leave 3.8 deprecated things in (but deprecated) until 3.10

Possibly there are many people who are moved or are just now moving to 3.7 from 3.6 and they are only just noticing these deprecated methods. Come June 1 they would be rudely removed. So maybe don't remove until 3.9.

We could discuss the policy on the list. Write it up on the wiki.

@telephon
Copy link
Member

telephon commented May 8, 2016

Yes, that seems good. we could add a field in the deprecated method, so that the code says when it will happen.

@timblechmann
Copy link
Contributor

might be a good idea to move this to a backwards-compatibility quark so that old code won't necessarily break. also, some of the methods didn't emit a warning, so it might be reasonable to keep them around for one more release (with a deprecation warning added)

@crucialfelix
Copy link
Member

We all agreed but nobody actually did it. They are removed in the 3.8 release (which will get merged into master)

Now master is the 3.9 track and I;ve proposed the same thing: remove deprecated 3.8 now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp: class library SC class library
Projects
None yet
Development

No branches or pull requests

5 participants