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

Configure codeclimate #259

Merged
merged 5 commits into from
Sep 1, 2020
Merged

Configure codeclimate #259

merged 5 commits into from
Sep 1, 2020
+4 −47

Conversation

rrjbca
Copy link
Contributor

@rrjbca rrjbca commented Sep 1, 2020

Description

Configure codeclimate to enforce pep8 style only:

  • Configure pycodestyle in the [pep8] section of setup.cfg (see https://docs.codeclimate.com/docs/pep8)
  • Disable the codestyle GitHub Action previously used to run flake8 and enforce code style
  • Do not upload test coverage reports to codeclimate (codecov is already used for reporting test coverage)

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 September 1, 2020 14:58
@rrjbca rrjbca merged commit b48fe48 into skypyproject:master Sep 1, 2020
@rrjbca rrjbca deleted the configure_codeclimate branch September 1, 2020 15:27
rrjbca added a commit to rrjbca/skypy that referenced this pull request Feb 16, 2021
@rrjbca
Revert "Configure codeclimate (skypyproject#259)"
96c2a6a 
This reverts commit b48fe48.
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
continuous integration
Projects
None yet
Milestone
No milestone
Development

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