Skip to content
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

Update monkeypatch.rst #230

Merged
merged 2 commits into from
Dec 3, 2013
Merged

Update monkeypatch.rst #230

merged 2 commits into from
Dec 3, 2013

Conversation

marcosfromero
Copy link
Contributor

No description provided.

@hvelarde
Copy link
Member

hvelarde commented Dec 2, 2013

please keep line length to 72 characters.

from PEP 8: For flowing long blocks of text with fewer structural restrictions (docstrings or comments), the line length should be limited to 72 characters.

@svx svx mentioned this pull request Dec 2, 2013
@marcosfromero
Copy link
Contributor Author

I thought it was only for Python code, not RST documentation.
Should I also change the original lines (first paragraph) of the file?

@hvelarde
Copy link
Member

hvelarde commented Dec 2, 2013

I was looking for a reference on documentation but I was not able to find one, that's why I used the one on PEP 8; anyway, the idea is not having a paragraph formated as a line.

yes, please fix any other line on the file with the same issue.

Kept text lines up to 72 characters
hvelarde added a commit that referenced this pull request Dec 3, 2013
@hvelarde hvelarde merged commit 18f1950 into collective:master Dec 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants