Refactor integrations to format with ruff after dropping black #20452
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
datadog-agent-integrations-bot
bot
added
integration/btrfs
integration/couchbase
integration/marathon
dev/tooling
integration/http_check
integration/apache
integration/aspdotnet
integration/cacti
integration/cassandra_nodetool
integration/ceph
integration/consul
integration/directory
integration/disk
integration/dns_check
integration/dotnetclr
integration/ecs_fargate
integration/elastic
integration/envoy
integration/etcd
integration/gearmand
integration/gunicorn
integration/haproxy
integration/iis
integration/istio
integration/kafka_consumer
integration/kong
integration/kube_dns
integration/kube_proxy
integration/kubelet
integration/kubernetes_state
labels
iliakur
reviewed
Jun 5, 2025
zoedt
reviewed
Jun 5, 2025
zoedt
approved these changes
Jun 5, 2025
estherk15
reviewed
Jun 5, 2025
AAraKKe
force-pushed
the
aarake/format-all-integrations-with-ruff
branch
from
af68987 to
808d1a8
Compare
AAraKKe
force-pushed
the
aarake/small-tweaks-for-devx
branch
from
cb7923e to
73c496a
Compare
jedupau
approved these changes
Jun 6, 2025
AAraKKe
force-pushed
the
aarake/small-tweaks-for-devx
branch
from
73c496a to
d9c7f24
Compare
AAraKKe
force-pushed
the
aarake/format-all-integrations-with-ruff
branch
2 times, most recently
from
60841c2 to
12ea7d9
Compare
AAraKKe
force-pushed
the
aarake/small-tweaks-for-devx
branch
from
d9c7f24 to
38b8853
Compare
AAraKKe
force-pushed
the
aarake/format-all-integrations-with-ruff
branch
2 times, most recently
from
5ce7d14 to
cadf75c
Compare
AAraKKe
force-pushed
the
aarake/small-tweaks-for-devx
branch
from
38b8853 to
962481f
Compare
AAraKKe
force-pushed
the
aarake/format-all-integrations-with-ruff
branch
from
cadf75c to
3d8e1da
Compare
AAraKKe
force-pushed
the
aarake/small-tweaks-for-devx
branch
from
962481f to
a3a8e0d
Compare
estherk15
approved these changes
Jun 6, 2025
|
/mergegate get |
|
View all feedbacks in Devflow UI.
Gate status is success for commit 3d8e1da |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This PR runs ruff formatter in all integrations following the previous PR #20451.
Motivation
While Ruff is mostly a drop in replacement of Black there are some rules for which ruff can decide to format things different. See their differences with black page here.
Ruff brings faster formatting than Black and more reasonable defaults in many cases but this means that due to the differences in some situations, formatting would be a bit different.
Review checklist (to be filled by reviewers)
qa/skip-qalabel if the PR doesn't need to be tested during QA.backport/<branch-name>label to the PR and it will automatically open a backport PR once this one is merged