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

Update pylintrc #2951

Merged
merged 6 commits into from
Jan 10, 2024
Merged

Conversation

cmintey
Copy link
Contributor

@cmintey cmintey commented Jan 10, 2024

What type of PR is this?

  • cleanup

What this PR does / why we need it:

Removes useless pylint options from disabled that were spamming VSCode with warning messages. Also changed some pylint options to be a little more reasonable. Now there are a lot less pylint issues, but there are still a ton. This will hopefully make it easier to identity actual code smells

hay-kot
hay-kot previously approved these changes Jan 10, 2024
@michael-genson michael-genson enabled auto-merge (squash) January 10, 2024 17:40
Copy link
Collaborator

@michael-genson michael-genson left a comment

Choose a reason for hiding this comment

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

had to fix the lockfile because of all the other dependency PRs, but it should work now

@michael-genson michael-genson merged commit 53b790b into mealie-recipes:mealie-next Jan 10, 2024
9 checks passed
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.

4 participants