Skip to content

Commit

Permalink
Removes pylint from our toolchain (apache/airflow#16682)
Browse files Browse the repository at this point in the history
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

Partial Commit Extracted From: https://github.com/apache/airflow
  • Loading branch information
potiuk authored and cognifloyd committed Feb 16, 2022
1 parent 37877e9 commit ca283f7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/helm_template_generator.py
Expand Up @@ -33,7 +33,7 @@
BASE_URL_SPEC = "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.15.0"

crd_lookup = {
'keda.sh/v1alpha1::ScaledObject': 'https://raw.githubusercontent.com/kedacore/keda/v2.0.0/config/crd/bases/keda.sh_scaledobjects.yaml', # noqa: E501 # pylint: disable=line-too-long
'keda.sh/v1alpha1::ScaledObject': 'https://raw.githubusercontent.com/kedacore/keda/v2.0.0/config/crd/bases/keda.sh_scaledobjects.yaml', # noqa: E501
}


Expand Down Expand Up @@ -120,4 +120,4 @@ def render_k8s_object(obj, type_to_render):
"""
Function that renders dictionaries into k8s objects. For helm chart testing only.
"""
return api_client._ApiClient__deserialize_model(obj, type_to_render) # pylint: disable=W0212
return api_client._ApiClient__deserialize_model(obj, type_to_render)

0 comments on commit ca283f7

Please sign in to comment.