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

CI/CD: Format Python Code #1318

Closed
deadex-ng opened this issue Jan 10, 2023 · 5 comments · Fixed by #1646
Closed

CI/CD: Format Python Code #1318

deadex-ng opened this issue Jan 10, 2023 · 5 comments · Fixed by #1646

Comments

@deadex-ng
Copy link

deadex-ng commented Jan 10, 2023

Is your feature request related to a problem? Please describe.
We're receiving python code from different contributors, which means different developers format the code differently.

Describe the solution you'd like
We should have a standard format for the code. I suggest we use black
We can have a workflow for this. Once a contributor opens a PR, we check the code format.

Describe alternatives you've considered
I suggest we use black. It's a very popular code formatter.

Approach to be followed (optional)
We can have a workflow for this. Once a contributor opens a PR, we check the code format.

Additional context
I would like to work on this issue for SWC season 3

@deadex-ng deadex-ng added the enhancement New feature or request label Jan 10, 2023
@deadex-ng deadex-ng changed the title Feature request CI/CD: Format Python Code Jan 10, 2023
@github-actions
Copy link

Hello @deadex-ng, Thank you for generating an issue to this project! Please wait while we get back to you.

@deadex-ng
Copy link
Author

@akshitagupta15june I'd like to work on this.

@rafiya2003
Copy link

I would like to contribute in it

@elle-Bee
Copy link
Contributor

Hey @akshitagupta15june can I work on this issue under the SSOC?

@Garvanand
Copy link

@akshitagupta15june please assign this issue to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants