Skip to content

Assign each of our pytest warnings to the algorithm authors #7211

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

Closed
1 task
cclauss opened this issue Oct 15, 2022 · 5 comments
Closed
1 task

Assign each of our pytest warnings to the algorithm authors #7211

cclauss opened this issue Oct 15, 2022 · 5 comments
Labels
enhancement This PR modified some existing files help wanted

Comments

@cclauss
Copy link
Member

cclauss commented Oct 15, 2022

Feature description

Pytest generates a lot of warnings that we should get resolved. https://github.com/TheAlgorithms/Python/actions/runs/3254565205/jobs/5342967879#step:6:577

Go through each warning and look at the Python file to see who created it or modified it. Create an issue asking the author(s) to fix the warning. If they do not respond with a fix within seven days then find someone in the community that is willing to propose a fix. Let’s get the warnings back down to zero but let’s give the authors the first crack at proposing fixes.

Would you like to work on this feature?

  • Yes, I want to work on this feature!
@cclauss cclauss added enhancement This PR modified some existing files help wanted labels Oct 15, 2022
@cclauss cclauss changed the title Assign each of our pytest warnings to the algorithm author Assign each of our pytest warnings to the algorithm authors Oct 15, 2022
@0xSumitBanik
Copy link

So, we would be analyzing the warnings and create the issues manually.
Am I right?

@CaedenPH
Copy link
Contributor

CaedenPH commented Oct 16, 2022

@cclauss Is this what you meant?

I don't have permissions to assign, so I just pinged

@PARVASHWANI
Copy link

Yes I would like to work on this feature.

@mahak-dev
Copy link

hello sir,
I want to solve this problem, I have a little bit of knowledge regarding on this Topic

@cclauss
Copy link
Member Author

cclauss commented Oct 19, 2022

@PARVASHWANI and @mahak-dev Why are you waiting? If you see something that you can fix then open a pull request.

@cclauss cclauss closed this as completed Oct 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This PR modified some existing files help wanted
Projects
None yet
Development

No branches or pull requests

5 participants