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

Add VsCode support for pydoclint #4241

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

cidrblock
Copy link
Contributor

@cidrblock cidrblock commented Jun 26, 2024

Since flake8 has been removed, VsCode no longer provided feedback during development about new docstring errors.

This PR add a new task called pydoclint that runs pydoclint in the background. Message from pydoclint are parsed with a regex and presented in the problems tab of VsCode by the task.

VsCode does not have native support for running a task on save, so a new extension is added to support that.

The flake8 extension is removed which was an oversight in #4235

Related #4236

@cidrblock
Copy link
Contributor Author

Gonna merge this one so it can be carried over to other projects

@cidrblock cidrblock merged commit 4fe7993 into ansible:main Jun 26, 2024
17 checks passed
cidrblock added a commit that referenced this pull request Jun 26, 2024
- finish removing flake8
- update constraints after flake8 removal
- update pydoclint config and update baseline
Followup work from #4241 and #4235
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant