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

Removed a simple pitfall case of a consider-using-in case #230

Open

NaelsonDouglas wants to merge 1 commit into Morizeyao:old_gpt_2_chinese_before_2021_4_22 from python-pitfalls:old_gpt_2_chinese_before_2021_4_22

NaelsonDouglas commented Nov 2, 2021

Problem:
The code was incurring into the consider-using-in pitfall case, as described by Pylint documentation here https://github.com/vald-phoenix/pylint-errors/blob/master/plerr/errors/refactoring/R1714.md

Solution:
Applied adequate refactoring. Only a single line of code was changed


          Removed a simple pitfall case of a consider-using-in case

1ac4a3b

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