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

Refactor pylint/config modules to avoid using deprecated optparse module. #4455

Closed
matusvalo opened this issue May 9, 2021 · 1 comment
Closed

Refactor pylint/config modules to avoid using deprecated optparse module. #4455

matusvalo opened this issue May 9, 2021 · 1 comment
Labels
Command line Related to command line interface Enhancement ✨ Improvement to a component High effort 🏋 Difficult solution or problem to solve Maintenance Discussion or action around maintaining pylint or the dev workflow

Comments

@matusvalo
Copy link
Collaborator

Steps to reproduce

Following files:

  • pylint/config/man_help_formatter.py
  • pylint/config/option.py
  • pylint/config/option_manager_mixin.py
  • pylint/config/option_parser.py
  • pylint/config/options_provider_mixin.py

Are using deprecated optparse module. Currently deprecated-module check is disabled to suppress errors. When disable is removed, pylint is raising error for given modules.

Current behavior

optparse module is used in mentioned modules, deprecated-module check is disabled to suppress errors.

Expected behavior

Modules should be refactored to avoid using optparse module. deprecated-module check should not be disabled.
@matusvalo matusvalo mentioned this issue May 9, 2021
Merged
4 tasks
@Pierre-Sassoulas Pierre-Sassoulas added Enhancement ✨ Improvement to a component task Command line Related to command line interface labels May 10, 2021
@Pierre-Sassoulas Pierre-Sassoulas mentioned this issue Oct 25, 2021
Closed
@Pierre-Sassoulas Pierre-Sassoulas added the High effort 🏋 Difficult solution or problem to solve label Oct 25, 2021
@DanielNoord
Copy link
Collaborator

Closing this in favour of #5392.

@Pierre-Sassoulas Pierre-Sassoulas added Maintenance Discussion or action around maintaining pylint or the dev workflow and removed task labels Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Command line Related to command line interface Enhancement ✨ Improvement to a component High effort 🏋 Difficult solution or problem to solve Maintenance Discussion or action around maintaining pylint or the dev workflow
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@matusvalo @Pierre-Sassoulas @DanielNoord