-
Hello, I am trying to make a PR while following the guidelines
All done! ✨ 🍰 ✨
201 files would be left unchanged.
[some output here]
************* Module safety_gymnasium.bases.underlying
safety_gymnasium/bases/underlying.py:265:0: W0621: Redefining name 'geoms' from outer scope (line 30) (redefined-outer-name)
safety_gymnasium/bases/underlying.py:275:0: W0621: Redefining name 'free_geoms' from outer scope (line 29) (redefined-outer-name)
safety_gymnasium/bases/underlying.py:285:0: W0621: Redefining name 'mocaps' from outer scope (line 31) (redefined-outer-name)
------------------------------------------------------------------
Your code has been rated at 9.99/10 (previous run: 9.99/10, +0.00)
make: *** [Makefile:105: pylint] Error 4
pylint...................................................................Failed
- hook id: pylint
- exit code: 4
************* Module safety_gymnasium.bases.underlying
safety_gymnasium/bases/underlying.py:265:0: W0621: Redefining name 'geoms' from outer scope (line 30) (redefined-outer-name)
safety_gymnasium/bases/underlying.py:275:0: W0621: Redefining name 'free_geoms' from outer scope (line 29) (redefined-outer-name)
safety_gymnasium/bases/underlying.py:285:0: W0621: Redefining name 'mocaps' from outer scope (line 31) (redefined-outer-name)
------------------------------------------------------------------
Your code has been rated at 9.97/10 (previous run: 9.97/10, +0.00) I am wondering if I am doing something wrong or running |
Beta Was this translation helpful? Give feedback.
Answered by
muchvo
Jan 9, 2024
Replies: 1 comment
-
No worries, these commands are mainly to ensure consistent code style. Feel free to submit your PR first, and I'll take care of these issues before we finalize the merge. |
Beta Was this translation helpful? Give feedback.
0 replies
Answer selected by
aivarsoo
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No worries, these commands are mainly to ensure consistent code style. Feel free to submit your PR first, and I'll take care of these issues before we finalize the merge.