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.

Sign up for GitHub

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

Jump to bottom

Pylint fix #21

Open

chiefeu wants to merge 6 commits into jtoy:main from chiefeu:pylint-fix

Contributor

chiefeu commented Mar 8, 2024

initially used 'black' (pip install black) for basic fix.
then manually modified each python file based on the pylint output.

added module, class, function docstrings
removed/commented out unused variables (in for loops, we rename the variable to _ )
removed/commented out unused imports
reorganized imports
fixed line-too-long warnings
fixed naming conventions suggested by pylint
fixed missing explicit default parameters
etc.

other pylint warnings (too many branch, local variables, statements, etc.) needs to refactor the whole code to be fixed

src python files for further refactoring.

chiefeu and others added 6 commits

March 5, 2024 22:34


          Updated Sidebar.tsx

3f943a5

Show the history of talking and thinking and add a button to close the window.


          added mouse pointer

90dc2d4

https://trello.com/c/ZhNY6kKG/301-hovering-over-someone-should-show-a-mouse-pointer


          Merge branch 'jtoy:main' into main

c4a5817


          fix pylint errors 1

5d18b3f


          fix pylint errors 1

fe60dd3


          fixed errors for test_matrix_runs_step

2edaa76

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

Labels

None yet