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

Fixes spelling in Map #2730

Merged
merged 3 commits into from Aug 22, 2018

Conversation

Projects
None yet
4 participants
@wafels
Copy link
Member

commented Aug 16, 2018

The map default heliographic latitude variable was misspelled - see issue #2716. This PR fixes #2716.

@pep8speaks

This comment has been minimized.

Copy link

commented Aug 16, 2018

Hello @wafels! Thanks for updating the PR.

Line 147:101: E501 line too long (104 > 100 characters)
Line 790:13: E129 visually indented line with same indent as next logical line
Line 824:101: E501 line too long (101 > 100 characters)
Line 1212:101: E501 line too long (108 > 100 characters)
Line 1213:101: E501 line too long (106 > 100 characters)
Line 1266:101: E501 line too long (103 > 100 characters)
Line 1301:101: E501 line too long (119 > 100 characters)
Line 1370:101: E501 line too long (139 > 100 characters)
Line 1413:101: E501 line too long (128 > 100 characters)
Line 1414:101: E501 line too long (128 > 100 characters)
Line 1455:101: E501 line too long (121 > 100 characters)

Comment last updated on August 22, 2018 at 09:11 Hours UTC
@sunpy-bot

This comment has been minimized.

Copy link

commented Aug 16, 2018

Thanks for the pull request @wafels! Everything looks great!

@wafels wafels self-assigned this Aug 16, 2018

@Cadair Cadair added this to the 0.9.3 milestone Aug 16, 2018

@Cadair Cadair modified the milestones: 0.9.3, 1.0 Aug 16, 2018

@Cadair Cadair self-requested a review Aug 16, 2018

@Cadair

Cadair approved these changes Aug 16, 2018

wafels and others added some commits Aug 16, 2018

@Cadair Cadair force-pushed the wafels:badspellinginmap branch from 845f5d9 to cac5105 Aug 22, 2018

@Cadair Cadair merged commit 652c34a into sunpy:master Aug 22, 2018

4 checks passed

ci/circleci: egg-info-36 Your tests passed on CircleCI!
Details
ci/circleci: figure-tests-36 Your tests passed on CircleCI!
Details
ci/circleci: html-docs Your tests passed on CircleCI!
Details
sunpy-bot All checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.