Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Reorganize the developer docs #1512

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

Merged
pelson merged 4 commits into from
Nov 20, 2012
Merged

Reorganize the developer docs #1512

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
8ec5b2c
Reorganize the developer docs. Most importantly, this adds a pull re…
mdboom Nov 16, 2012
050b27f
Fix release guide
mdboom Nov 16, 2012
4f75fff
Update coding guide with suggestions from the PR.
mdboom Nov 16, 2012
6dda1e2
More minor fixes based on feedback in the PR.
mdboom Nov 19, 2012
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update coding guide with suggestions from the PR.
  • Loading branch information
@mdboom
mdboom committed Nov 16, 2012
commit 4f75fff0bd8e7a1382d19a20b60d9b195589cdab
9 changes: 8 additions & 1 deletion doc/devel/coding_guide.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ Style
- You may want to consider installing automatic PEP8 checking in
your editor.

* No tabs (only space). No trailing whitespace.
* No tabs (only spaces). No trailing whitespace.

- Configuring your editor to remove these things upon saving will
save a lot of trouble.
Expand All @@ -49,11 +49,16 @@ Style
import matplotlib as mpl
from matplotlib import pyplot as plt
import matplotlib.cbook as cbook
Copy link
Copy Markdown
Member

@WeatherGod WeatherGod Nov 16, 2012

Choose a reason for hiding this comment

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

import matplotlib.collections as mcol
import matplotlib.patches as mpatches

or whatever it is that is standard...

Copy link
Copy Markdown
Member Author

@mdboom mdboom Nov 16, 2012

Choose a reason for hiding this comment

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

I'll add these. I've never been a huge fan of that, but it's sometimes required, and of course is used a lot throughout the codebase. We can remove them here if we clean that style up globally later.

import matplotlib.collections as mcol
import matplotlib.patches as mpatches

* See below for additional points about
:ref:`keyword-argument-processing`, if code in your pull request
does that.

* Adding a new pyplot function involves generating code. See
:ref:`new-pyplot-function` for more information.

Documentation
-------------

Expand Down Expand Up @@ -236,6 +241,8 @@ example code you can load it into a file handle with::
import matplotlib.cbook as cbook
fh = cbook.get_sample_data('mydata.dat')

.. _new-pyplot-function:

Writing a new pyplot function
-----------------------------

Expand Down