Skip to content

[SDK-3182] Add some code style tools#311

Merged
adamjmcgrath merged 2 commits intomasterfrom
code-etyle
Apr 1, 2022
Merged

[SDK-3182] Add some code style tools#311
adamjmcgrath merged 2 commits intomasterfrom
code-etyle

Conversation

@adamjmcgrath
Copy link
Copy Markdown
Contributor

Changes

Add a bunch of pre commit hooks for PEP8 and code style: flake8, isort and black

All the code changes are the result of running black

@adamjmcgrath adamjmcgrath added the review:medium Medium review label Mar 25, 2022
@adamjmcgrath adamjmcgrath requested a review from a team as a code owner March 25, 2022 18:13
@adamjmcgrath adamjmcgrath marked this pull request as draft March 25, 2022 18:21
@adamjmcgrath adamjmcgrath force-pushed the code-etyle branch 3 times, most recently from 986a50d to f331be3 Compare March 29, 2022 08:55
@adamjmcgrath adamjmcgrath changed the title Add some code style tools [SDK-3182] Add some code style tools Mar 29, 2022
@adamjmcgrath adamjmcgrath marked this pull request as ready for review March 29, 2022 09:27
Copy link
Copy Markdown
Contributor

@evansims evansims left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fantastic! Love to see code styling getting applied 👍 So much more readable

@adamjmcgrath adamjmcgrath merged commit f15520b into master Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review:medium Medium review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants