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.

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

clean pylint issue for legacy code of autorest.python #2994

Open
msyyc opened this issue Jan 7, 2025 · 2 comments
Open

clean pylint issue for legacy code of autorest.python #2994

msyyc opened this issue Jan 7, 2025 · 2 comments
Assignees
@msyyc
Labels
mq This issue is part of a "milestone of quality" initiative.

Comments

@msyyc
Copy link
Member

msyyc commented Jan 7, 2025

There is still some pylint issue in legacy code of autorest.python folder: #2981 (comment).
We shall remove the disable configuration of pylintrc after fix them.
@msyyc msyyc added the mq This issue is part of a "milestone of quality" initiative. label Jan 7, 2025
@msyyc msyyc self-assigned this Jan 7, 2025
@iscai-msft
Copy link
Contributor

is this legacy code in the autorest.python code package, or legacy SDKs generated by autorest.python. I'm fine to add pylint checks for legacy code SDKs generated as well

@msyyc
Copy link
Member Author

msyyc commented Jan 10, 2025

is this legacy code in the autorest.python code package, or legacy SDKs generated by autorest.python. I'm fine to add pylint checks for legacy code SDKs generated as well

legacy SDKs generated by autorest.python

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

@msyyc msyyc

Labels
mq This issue is part of a "milestone of quality" initiative.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@iscai-msft @msyyc