Skip to content

Conversation

@pankajagrawal16
Copy link
Contributor

@pankajagrawal16 pankajagrawal16 commented Mar 15, 2021

Issue #, if available:

Description of changes:

Checklist

Breaking change checklist

RFC issue #:

  • Migration process documented
  • Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@pankajagrawal16 pankajagrawal16 requested a review from msailes March 15, 2021 10:43
@msailes
Copy link
Contributor

msailes commented Mar 23, 2021

So this has been deprecated since Jan, would the plan be to document the migration in the release notes?

Which part of the version number do you think we should change?

@pankajagrawal16
Copy link
Contributor Author

So this has been deprecated since Jan, would the plan be to document the migration in the release notes?

Which part of the version number do you think we should change?

Yes, the plan is to document it as part of the release note and change log. I am still considering a minor version increase since this should cause compile time issue for users

@pankajagrawal16
Copy link
Contributor Author

@msailes Have a look and see if you agree :)

Copy link
Contributor

@msailes msailes left a comment

Choose a reason for hiding this comment

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

👍

@pankajagrawal16 pankajagrawal16 merged commit 860aad9 into master Mar 25, 2021
@pankajagrawal16 pankajagrawal16 deleted the remove-deprecated-cature-mode branch March 25, 2021 11:24
pankajagrawal16 added a commit that referenced this pull request Mar 30, 2021
pankajagrawal16 added a commit that referenced this pull request Mar 30, 2021
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.

2 participants