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

using compat-lifecycle #4036

Merged
merged 6 commits into from
Dec 24, 2018
Merged

using compat-lifecycle #4036

merged 6 commits into from
Dec 24, 2018

Conversation

romainfrancois
Copy link
Member

@lionel- would you have a look please ?

@lionel-
Copy link
Member

lionel- commented Dec 20, 2018

I think it's too harsh to deprecate the underscore functions without a soft-deprecation cycle. Also should we retire them instead of soft-deprecating them? i.e. maintain these functions undefinitely as long as they are not a maintenance burden.

@romainfrancois
Copy link
Member Author

Were they not soft deprecated already ?

@lionel-
Copy link
Member

lionel- commented Dec 20, 2018

Were they not soft deprecated already ?

Not in the compat-lifecycle sense. They did not warn users calling from the global env nor developers testing their packages. These soft warnings are meant to ease the deprecation process.

@romainfrancois
Copy link
Member Author

I see. I'll move them to soft deprecated then. Does that mean #4043 should be reopen ?

@hadley
Copy link
Member

hadley commented Dec 20, 2018

No, because they are soft deprecated, and we should be steering people away from them.

@romainfrancois romainfrancois merged commit bee3351 into master Dec 24, 2018
@romainfrancois romainfrancois deleted the lifecycle branch December 24, 2018 08:07
@lock
Copy link

lock bot commented Jun 22, 2019

This old issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with reprex) and link to this issue. https://reprex.tidyverse.org/

@lock lock bot locked and limited conversation to collaborators Jun 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants