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

Add pep8 check #1131

Closed
wants to merge 5 commits into from
Closed

Add pep8 check #1131

wants to merge 5 commits into from

Conversation

mtazzari
Copy link
Contributor

@mtazzari mtazzari commented Oct 20, 2022

This PR adds PEP8 check to the whole repository and Fix #1132 .

Release notes feature title

... Release notes description / summary
... Any text between these two tags will be automatically pulled into the platform release notes

@mtazzari mtazzari changed the base branch from master to develop October 20, 2022 14:23
@mtazzari mtazzari linked an issue Oct 20, 2022 that may be closed by this pull request
@mtazzari mtazzari requested a review from sambles October 20, 2022 15:52
@mtazzari mtazzari marked this pull request as ready for review October 20, 2022 15:52
@mtazzari
Copy link
Contributor Author

The GH actions workflow is ready to be deployed, but since essentially the whole codebase is not PEP8 compliant, it will fail until we do a whole pass of the codebase.
I will close the PR for now and we resume it when we've done the PEP8 pass.

@mtazzari mtazzari closed this Oct 20, 2022
@mtazzari mtazzari self-assigned this Oct 20, 2022
@mtazzari mtazzari deleted the feature/pep8-check branch December 9, 2022 10:29
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.

Make code PEP8 compliant
1 participant