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

Support globbing pattern for input specification #8310

Closed
alexey-pelykh opened this issue Feb 17, 2023 · 2 comments · Fixed by #8312
Closed

Support globbing pattern for input specification #8310

alexey-pelykh opened this issue Feb 17, 2023 · 2 comments · Fixed by #8312
Labels
Enhancement ✨ Improvement to a component Needs PR This issue is accepted, sufficiently specified and now needs an implementation
Milestone
2.17.0

Comments

@alexey-pelykh
Copy link
Contributor

alexey-pelykh commented Feb 17, 2023 •
edited
Loading

Current problem

As mentioned in #8281 (comment) and #8290 (comment), lets consider supporting pylint --recursive=y packages/*/src

cc @Pierre-Sassoulas @DudeNr33 @DanielNoord

Desired solution

Add globbing support right in this line
https://github.com/PyCQA/pylint/blob/dca394035268a234b29d0c103a4fcc201c84061f/pylint/config/config_initialization.py#L123

Additional context

No response
@alexey-pelykh alexey-pelykh added the Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling label Feb 17, 2023
This was referenced Feb 17, 2023
Merged
Closed
Merged
@Pierre-Sassoulas Pierre-Sassoulas added Enhancement ✨ Improvement to a component Needs PR This issue is accepted, sufficiently specified and now needs an implementation and removed Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Feb 17, 2023
@DudeNr33
Copy link
Collaborator

Note: although the example includes the --recursive=y option, globbing should work without it as well.

@alexey-pelykh
Copy link
Contributor Author

@Pierre-Sassoulas there's #8312 so the PR is there :D

@Pierre-Sassoulas Pierre-Sassoulas added this to the 2.17.0 milestone Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement ✨ Improvement to a component Needs PR This issue is accepted, sufficiently specified and now needs an implementation
Projects
None yet
Milestone
2.17.0
Development

Successfully merging a pull request may close this issue.

Support globbing pattern for input specification alexey-pelykh/pylint
3 participants
@alexey-pelykh @DudeNr33 @Pierre-Sassoulas