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

[qa] : Add Linters and Formatter #71

Closed
Aryamanz29 opened this issue Jun 14, 2021 · 4 comments · May be fixed by #78
Closed

[qa] : Add Linters and Formatter #71

Aryamanz29 opened this issue Jun 14, 2021 · 4 comments · May be fixed by #78
Labels
enhancement New feature or request good first issue Good for newcomers LGMSOC21

Comments

@Aryamanz29
Copy link
Collaborator

Aryamanz29 commented Jun 14, 2021

  • Need a workflow that ensures every contributor should follow the PEP-8 style guide for python code.
  • We can use flake8 for linting and Black for formatting source code.
  • Also this workflow runs on every PR which ensures the code follows PEP-8 standard or not.
@Aryamanz29 Aryamanz29 added enhancement New feature or request good first issue Good for newcomers LGMSOC21 level3 labels Jun 14, 2021
@chetak123
Copy link

can I work on this issue please?

@Aryamanz29
Copy link
Collaborator Author

can I work on this issue please?

Sure @chetak123 , If you stuck at any point feel free to ping me 😄
Happy Coding 👨‍💻

@Aryamanz29
Copy link
Collaborator Author

Aryamanz29 commented Jun 14, 2021

@chetak123 Before start working on this issue first please read contributing guidelines of this project.

@Aryamanz29
Copy link
Collaborator Author

@chetak123 Any updates??

@Samridhi-98 Samridhi-98 linked a pull request Jun 18, 2021 that will close this issue
5 tasks
@chetak123 chetak123 removed their assignment Jul 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers LGMSOC21
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants