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 pylint workflow and fix all issues #262

Open
djeck1432 opened this issue Oct 29, 2024 · 16 comments · May be fixed by #269
Open

Add pylint workflow and fix all issues #262

djeck1432 opened this issue Oct 29, 2024 · 16 comments · May be fixed by #269
Assignees
Labels
good first issue Good for newcomers ODHack9

Comments

@djeck1432
Copy link
Collaborator

  1. Read our Contributor Guidelines and README.
  2. Install pylint via poetry
  3. Run this command in apps folder
pylint $(git ls-files 'data_handler/*.py') --disable=all --enable=C0114,C0115,C0116,C0301
  1. Fix all issues which you will have in console
  2. Add github workflow to run this command for new PR:
pylint $(git ls-files 'data_handler/*.py') --disable=all --enable=C0114,C0115,C0116,C0301
@djeck1432 djeck1432 added good first issue Good for newcomers ODHack9 labels Oct 29, 2024
@binayak9932
Copy link
Contributor

I’m interested in this one.I think this is an easy task and i can work with it .and I love 🍍

@djeck1432
Copy link
Collaborator Author

@binayak9932 sorry, it's for new comers

@Kaminar-i
Copy link
Contributor

Can I be assigned this, I can start now

@djeck1432
Copy link
Collaborator Author

@Kaminar-i READ GUIDELINEEEEEEEEEEE

@mayasimi
Copy link
Contributor

I would like to take this task, I love 🍍 EDT 2days I am experienced with python and this would be my first contribution the derisk project kindly assign me

@djeck1432
Copy link
Collaborator Author

@mayasimi I will assign if you can do it during next 24 hours

@mayasimi
Copy link
Contributor

@mayasimi kindly assign I will get it done in 24hrs

@Akshola00
Copy link
Contributor

I’m interested in this one.

@djeck1432
Copy link
Collaborator Author

djeck1432 commented Oct 29, 2024

@Akshola00 sorry, tasks for new comers

@0xdevcollins
Copy link

Can I take care of this issue? I will deliver in 24 hrs

@djeck1432
Copy link
Collaborator Author

@0xdevcollins Nope, this task for new comers

@mimisavage
Copy link
Contributor

Could I take over this issue? I love 🍍
I will make i pr within 24hrs this is my first time working on this project

@sigmadawg
Copy link

I love 🍍 and im a newbie contributor, i would love to work on this.
Estimated timeframe : 10hrs

@suhas-sensei
Copy link

I would like to take this task, I love 🍍. To complete the pylint implementation task, I will install pylint through poetry, execute the provided linting command in the apps folder to identify and fix documentation and line length issues, then create a GitHub workflow file that runs the same pylint command on pull requests.
Timeframe : 1 day

@sigmadawg
Copy link

thank you for assigning @djeck1432

@djeck1432
Copy link
Collaborator Author

@sigmadawg If I won't see any draft PR with good progress ( not 2 line of code or something like that) You will be unassigned in 1.5 hour

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers ODHack9
Projects
None yet
9 participants