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

Fixed minor typos in documentation #1992

Merged
merged 5 commits into from
Mar 8, 2017
Merged

Fixed minor typos in documentation #1992

merged 5 commits into from
Mar 8, 2017

Conversation

nitinkgp23
Copy link
Contributor

This commit fixes minor typos in documentation contained in dev.rst

@nitinkgp23
Copy link
Contributor Author

@Cadair Why does the build fail? I have just fixed a typo.

@pep8speaks
Copy link

pep8speaks commented Feb 10, 2017

Hello @nitinkgp23! Thanks for updating the PR.

Line 33:101: E501 line too long (110 > 100 characters)
Line 34:15: E201 whitespace after '['
Line 34:101: E501 line too long (141 > 100 characters)
Line 34:140: E202 whitespace before ']'
Line 48:15: E201 whitespace after '['
Line 48:92: E202 whitespace before ']'
Line 56:101: E501 line too long (139 > 100 characters)
Line 80:25: E261 at least two spaces before inline comment
Line 86:101: E501 line too long (105 > 100 characters)
Line 110:40: E231 missing whitespace after ':'
Line 111:40: E231 missing whitespace after ':'
Line 115:40: E231 missing whitespace after ':'

Comment last updated on March 03, 2017 at 18:48 Hours UTC

@pep8speaks
Copy link

Hello @nitinkgp23! Thanks for updating the PR.

Line 445:101: E501 line too long (107 > 100 characters)
Line 628:94: W291 trailing whitespace
Line 630:92: W291 trailing whitespace

@pep8speaks
Copy link

Hello @nitinkgp23! Thanks for updating the PR.

This commit fixes minor typos in documentation contained in dev.rst
Due to merging of #1300, documentation didn't work as shown. This commit
fixes the bugs in documentation.
@Cadair Cadair merged commit 79a4689 into sunpy:master Mar 8, 2017
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.

None yet

4 participants