Skip to content

Can read setup.cfg and pyproject.toml files #3169

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

Merged
merged 1 commit into from
Oct 10, 2019
Merged

Can read setup.cfg and pyproject.toml files #3169

merged 1 commit into from
Oct 10, 2019

Conversation

AWhetter
Copy link
Contributor

@AWhetter AWhetter commented Oct 7, 2019

Steps

  • [*] Add yourself to CONTRIBUTORS if you are a new contributor.
  • [*] Add a ChangeLog entry describing what your PR does.
  • [*] If it's a new feature or an important bug fix, add a What's New entry in doc/whatsnew/<current release.rst>.
  • [*] Write a good description on what the PR does.

Description

Configuration can be read from a setup.cfg or pyproject.toml file. See the documentation for exactly how.

Type of Changes

Type
🐛 Bug fix
✨ New feature
🔨 Refactoring
📜 Docs

Related Issue

Closes #617

Copy link
Contributor

@PCManticore PCManticore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice and straightforward patch, thanks @AWhetter !

@@ -0,0 +1,27 @@
**************************
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks for adding one!

"astroid>=2.3.0,<2.4",
"isort>=4.2.5,<5",
"mccabe>=0.6,<0.7",
"toml",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a particular version that we should require here?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 89.607% when pulling 36fcf6f on fix_617 into 1e05190 on master.

@coveralls
Copy link

coveralls commented Oct 10, 2019

Coverage Status

Coverage decreased (-0.1%) to 89.625% when pulling 36fcf6f on fix_617 into 1e05190 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 89.607% when pulling 36fcf6f on fix_617 into 1e05190 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 89.607% when pulling 36fcf6f on fix_617 into 1e05190 on master.

@AWhetter AWhetter merged commit 3fc8c98 into master Oct 10, 2019
@AWhetter AWhetter deleted the fix_617 branch October 10, 2019 03:39
@flying-sheep
Copy link
Contributor

awesome, thank you!

kevinoid added a commit to kevinoid/pylint_runner that referenced this pull request Aug 25, 2021
Pylint 2.5.0 and later support reading configuration from
`pyproject.toml` or `setup.cfg` if `pylintrc` and `.pylintrc` do not
exist.  (pylint-dev/pylint#3169)  In order to support these configuration
files, use `pylint.lint.pylinter.PyLinter#read_config_file()` instead of
`configparser.ConfigParser#read()` to read the pylint configuration.

Signed-off-by: Kevin Locke <[email protected]>
kevinoid added a commit to kevinoid/pylint_runner that referenced this pull request Aug 26, 2021
Pylint 2.5.0 and later support reading configuration from
`pyproject.toml` or `setup.cfg` if `pylintrc` and `.pylintrc` do not
exist.  (pylint-dev/pylint#3169)  In order to support these configuration
files, use `pylint.lint.pylinter.PyLinter#read_config_file()` instead of
`configparser.ConfigParser#read()` to read the pylint configuration.

Signed-off-by: Kevin Locke <[email protected]>
kevinoid added a commit to kevinoid/pylint_runner that referenced this pull request Aug 26, 2021
Pylint 2.5.0 and later support reading configuration from
`pyproject.toml` or `setup.cfg` if `pylintrc` and `.pylintrc` do not
exist.  (pylint-dev/pylint#3169)  In order to support these configuration
files, use `pylint.lint.pylinter.PyLinter#read_config_file()` instead of
`configparser.ConfigParser#read()` to read the pylint configuration.

Signed-off-by: Kevin Locke <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Search pylint configuration in setup.cfg and pyproject.toml
4 participants