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

Removal of ini support in configuration #8418

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Pierre-Sassoulas
Copy link
Member

Type of Changes

Type
βœ“ πŸ”¨ Refactoring
βœ“ πŸ“œ Docs

Description

Reading comments it seems this was planned, but are we really doing it ? It seem the rst generated rely on the functions. What do you think @DanielNoord ?

@Pierre-Sassoulas Pierre-Sassoulas added Breaking changes for 3.0 🦀 Maintenance Discussion or action around maintaining pylint or the dev workflow labels Mar 9, 2023
@Pierre-Sassoulas Pierre-Sassoulas added this to the 3.0.0a6 milestone Mar 9, 2023
@Pierre-Sassoulas Pierre-Sassoulas marked this pull request as draft March 9, 2023 19:15
@DanielNoord
Copy link
Collaborator

I think this is blocked on pylint-config actually providing an updated and upgraded toml config file. It can generate a file based on current settings already but we want it to be a bit fancier.

After that I think most of this can be removed as there is no need to generate-rcfile anymore. I would still be in favour of deprecating the old command for 3.0 already as I think even pylint.toml is so much easier to parse as a format than the old ini formats.

@Pierre-Sassoulas
Copy link
Member Author

Removing from 3.0 then, we should deprecate and remove later.

@Pierre-Sassoulas Pierre-Sassoulas added the Blocked 🚧 Blocked by a particular issue label May 21, 2023
@Pierre-Sassoulas Pierre-Sassoulas changed the title Work in progress : Removal of ini support in configuration (?) Removal of ini support in configuration May 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked 🚧 Blocked by a particular issue Maintenance Discussion or action around maintaining pylint or the dev workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants