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
Add deprecation warnings for API changes #1005
Conversation
…here needed. I left in a backwards-compatible shim for giving the name argument to the LegendControl.
In starting to do the release, and looking over the changelog one last time, I felt there needed to be more explanation and consistency in our docs around #979. |
Also I realized #982 wasn't documented in the changelog, so I did that too. |
I created an 0.17.0 milestone to have the issues I think should be in the release. If those two issues pass review and are merged, I am happy to do the 0.17.0 release: |
@HaudinFlorence, you could also review this. Thanks again for the changes cleaning the api up! |
This also adds a top-level Releases category for the docs to give visibility to the changelog (and list of breaking changes, etc.) |
This also adds back in some API changes as deprecated, and switches a few more from PendingDeprecationWarnings to DeprecationWarnings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Add deprecation warnings for API changes
Add deprecation warnings for API changes
Add deprecation warnings for API changes
This is a followup of #979, correcting our LegendControl docs where needed.
I left in a backwards-compatible shim for giving the name argument to the LegendControl, since that is likely the most common thing people would have to change.