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
V2 1 test remove pylint #16878
V2 1 test remove pylint #16878
Conversation
rebuilding as some temporary failure happened |
The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease. |
I've opened #16882 so that we can keep this as two commits via GitHub UI. |
ca79a1d
to
491ddc8
Compare
06c2b81
to
ef7d0b0
Compare
#16682 was merged so I think this can be merged (with rebase) now? |
We've agreed during the voting process that Pylint support should be disabled: https://lists.apache.org/thread.html/r9e2cc385db8737ec0874ad09872081bd083593ee29e8303e58d21efb%40%3Cdev.airflow.apache.org%3E This PR: * removes all # pylint comments * removes pylint pre-commits and related scripts/files * removes CI jobs running pylint checks * removes documentation about pylint * removes unnecessary #noga (adds pre-commit for that) * fixes some remaining pydocstyle errors after removing #noqa's (cherry picked from commit 866a601)
c0fc1a8
to
064dd1c
Compare
Yep. Looks like :) merged |
No description provided.