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.

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

pylint config: builtins.Exception #16410

Merged
merged 1 commit into from
Sep 26, 2023
Merged

pylint config: builtins.Exception #16410

merged 1 commit into from
Sep 26, 2023

Conversation

altendky
Copy link
Contributor

@altendky altendky commented Sep 25, 2023

Purpose:

https://github.com/Chia-Network/chia-blockchain/actions/runs/6293301638/job/17086464327?pr=16407#step:8:13

pylint: Command line or configuration file:1: UserWarning: Specifying exception names in the overgeneral-exceptions option without module name is deprecated and support for it will be removed in pylint 3.0. Use fully qualified name (maybe 'builtins.Exception' ?) instead.

Current Behavior:

New Behavior:

Testing Notes:

https://github.com/Chia-Network/chia-blockchain/actions/runs/6293301638/job/17086464327?pr=16407#step:8:13
```
pylint: Command line or configuration file:1: UserWarning: Specifying exception names in the overgeneral-exceptions option without module name is deprecated and support for it will be removed in pylint 3.0. Use fully qualified name (maybe 'builtins.Exception' ?) instead.
```
@altendky altendky requested a review from a team as a code owner September 25, 2023 11:56
@altendky altendky added Tests Changes to tests Cleanup Code cleanup Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog labels Sep 25, 2023
@altendky altendky closed this Sep 25, 2023
@altendky altendky reopened this Sep 25, 2023
@wallentx wallentx merged commit fd4465a into main Sep 26, 2023
453 of 458 checks passed
@wallentx wallentx deleted the altendky-patch-18 branch September 26, 2023 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cleanup Code cleanup Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog Tests Changes to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants