8000 Minor refactor of backend_ps. by anntzer · Pull Request #10250 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]
Skip to content

Minor refactor of backend_ps. #10250

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 18, 2018
Merged

Minor refactor of backend_ps. #10250

merged 1 commit into from
Jan 18, 2018

Conversation

anntzer
Copy link
Contributor
@anntzer anntzer commented Jan 15, 2018 •
edited
Loading

PR Summary

Handles review comment that I missed in #10248 (comment).

Probably I should just merge _print_figure and _print_figure_tex at some point...

PR Checklist

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

@anntzer anntzer force-pushed the backend_ps-cleanup branch from d39de26 to 6f6f791 Compare January 15, 2018 07:23
@anntzer anntzer force-pushed the backend_ps-cleanup branch from 6f6f791 to 5bebf4b Compare January 15, 2018 19:28
@jklymak jklymak merged commit f02b34a into matplotlib:master Jan 18, 2018
@anntzer anntzer deleted the backend_ps-cleanup branch January 18, 2018 21:52
@QuLogic QuLogic added this to the v2.2.0 milestone Jul 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@QuLogic QuLogic QuLogic approved these changes

@jklymak jklymak jklymak approved these changes

Assignees
No one assigned
Labels
backend: ps
Projects
None yet
Milestone
v2.2.0
Development

Successfully merging this pull request may close these issues.
3 participants
@anntzer @QuLogic @jklymak
0