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.

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

Jump to bottom

Fix flake8 code style errors #428

Merged
merged 1 commit into from
Feb 16, 2021
Merged

Fix flake8 code style errors #428

merged 1 commit into from
Feb 16, 2021

Conversation

rrjbca
Copy link
Contributor

@rrjbca rrjbca commented Feb 16, 2021

Description

Fix style throughout the codebase such that flake8 skypy --max-line-length=100 returns no errors. This will allow us to use flake8 for our CI checks as in #427

Checklist

  • Follow the Contributor Guidelines
  • Write unit tests
  • Write documentation strings
  • Assign someone from your working team to review this pull request
  • Assign someone from the infrastructure team to review this pull request

@rrjbca rrjbca requested a review from a team February 16, 2021 11:14
@rrjbca rrjbca merged commit 3411b2e into skypyproject:master Feb 16, 2021
@rrjbca rrjbca deleted the fix-codestyle branch February 16, 2021 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ntessore ntessore ntessore approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rrjbca @ntessore