X Tutup
Skip to content

DOC: changed the documentation for layout_engine.py#24547

Closed
ramvikrams wants to merge 1 commit intomatplotlib:mainfrom
ramvikrams:t1
Closed

DOC: changed the documentation for layout_engine.py#24547
ramvikrams wants to merge 1 commit intomatplotlib:mainfrom
ramvikrams:t1

Conversation

@ramvikrams
Copy link
Copy Markdown
Contributor

PR Summary

closes #24518
changed the layout_engine documentation as specified in the issue
Additional just ran black on the lib/matplotlib/layout_engine.py just to modify the code for it becoming easier in readability

PR Checklist

Documentation and Tests

  • [N/A ] Has pytest style unit tests (and pytest passes)
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • [N/A ] New plotting related features are documented with examples.

Release Notes

  • [N/A ] New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • [ N/A] API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • [ N/A] Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

@ramvikrams ramvikrams closed this Nov 26, 2022
@ramvikrams ramvikrams deleted the t1 branch November 26, 2022 15:51
Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for opening your first PR into Matplotlib!

If you have not heard from us in a while, please feel free to ping @matplotlib/developers or anyone who has commented on the PR. Most of our reviewers are volunteers and sometimes things fall through the cracks.

You can also join us on gitter for real-time discussion.

For details on testing, writing docs, and our review process, please see the developer guide

We strive to be a welcoming and open project. Please follow our Code of Conduct.

@tacaswell
Copy link
Copy Markdown
Member

Was replaced by #24548

@ramvikrams
Copy link
Copy Markdown
Contributor Author

Was replaced by #24548

yes, thanks

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.

[Doc]: layout_engine description

3 participants

X Tutup