Skip to content

Fix cross-references in API changes < 2.0.0. #15070

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

Merged
merged 1 commit into from
Aug 18, 2019

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented Aug 17, 2019

PR Summary

More of the same. No backport since it changes references to code-style for deleted targets.

PR Checklist

  • [N/A] Has Pytest style unit tests
  • [N/A] Code is Flake 8 compliant
  • [N/A] New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • [N/A] Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • [N/A] Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

@QuLogic QuLogic added this to the v3.2.0 milestone Aug 17, 2019
- *xy* -> set where the arrow points to
- *anncoords* -> set the units of the annotation location
- *xycoords* -> set the units of the point location
- ``set_position()`` -> `.Annotation` only set location of annotation
Copy link
Contributor

Choose a reason for hiding this comment

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

'.Annotation.set_position'? (quotes -> backticks)

Copy link
Member Author

Choose a reason for hiding this comment

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

There is no .Annotation.set_position; it's on the underlying Text. But this is referring to the Annotation, and the fact that .set_position now works on it correctly, so I'm not sure we want to redirect to Text.

Tightened validation so that only {'tip', 'tail', 'mid', and 'middle'} (but any
capitalization) are valid values for the *pivot* keyword argument in the
`.Quiver` class (and hence `.axes.Axes.quiver` and `.pyplot.quiver` which both
fully delegate to `.Quiver`). Previously any input matching 'mid.*' would be
Copy link
Contributor

Choose a reason for hiding this comment

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

needs quoting to avoid asterisk being interpreted as italic (just below too)

Copy link
Member Author

Choose a reason for hiding this comment

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

That's only true on GitHub; it's correct in the rendered result.

@anntzer anntzer merged commit ddd23a8 into matplotlib:master Aug 18, 2019
@QuLogic QuLogic deleted the fix-xref-whats-new-2 branch August 18, 2019 17:46
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.

2 participants