Drop Black as formater from the lint environment in ddev #20451

AAraKKe · 2025-06-05T09:44:05Z

What does this PR do?

This PR removes the black dependency in the environment collector to allow

Motivation

For some reason we are not using ruff to format and just to lint our code. However, when running in an IDE developers might be using ruff to format on save which is efficient and can be configure in the same way as black. However, quotes follow by default PEP guidelines using double quotes which means that by formatting with ruff from the IDE (like VSCode) string quotes can be modified by mistake.

The current black configuration includes ignoring string quotes but the ruff configuration does not. This PR updates it.

The pydantic version change is done to be in line with the base package.

Review checklist (to be filled by reviewers)

Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

AAraKKe · 2025-06-05T09:44:23Z

This stack of pull requests is managed by Graphite. Learn more about stacking.

codecov · 2025-06-05T09:46:50Z

Codecov Report

Attention: Patch coverage is 30.76923% with 9 lines in your changes missing coverage. Please review.

Project coverage is 89.42%. Comparing base (4d7826b) to head (a3a8e0d).

Additional details and impacted files

Flag	Coverage Δ
active_directory	`93.18% <ø> (ø)`
activemq	`52.80% <ø> (ø)`
activemq_xml	`82.20% <ø> (ø)`
amazon_msk	`89.50% <ø> (ø)`
ambari	`85.75% <ø> (ø)`
appgate_sdp	`93.93% <ø> (ø)`
argo_rollouts	`90.00% <ø> (ø)`
argocd	`87.16% <ø> (ø)`
avi_vantage	`93.83% <ø> (ø)`
boundary	`100.00% <ø> (ø)`
cassandra	`66.66% <ø> (ø)`
celery	`95.45% <ø> (ø)`
cilium	`77.92% <ø> (ø)`
clickhouse	`94.55% <ø> (ø)`
confluent_platform	`81.96% <ø> (ø)`
couch	`94.76% <ø> (ø)`
crio	`89.79% <ø> (ø)`
datadog_cluster_agent	`90.19% <ø> (ø)`
dcgm	`93.54% <ø> (ø)`
ddev	`87.95% <30.76%> (-0.07%)`	⬇️
druid	`97.70% <ø> (ø)`
eks_fargate	`94.05% <ø> (ø)`
exchange_server	`93.06% <ø> (ø)`
external_dns	`89.28% <ø> (ø)`
fluentd	`84.21% <ø> (ø)`
fluxcd	`88.31% <ø> (ø)`
gitlab	`92.27% <ø> (ø)`
gitlab_runner	`92.76% <ø> (ø)`
go_expvar	`92.66% <ø> (ø)`
harbor	`89.08% <ø> (ø)`
hazelcast	`92.30% <ø> (ø)`
hdfs_datanode	`89.63% <ø> (ø)`
hdfs_namenode	`86.60% <ø> (ø)`
hive	`51.42% <ø> (ø)`
hivemq	`61.90% <ø> (ø)`
hudi	`73.91% <ø> (ø)`
ibm_ace	`92.25% <ø> (ø)`
ignite	`46.66% <ø> (ø)`
jboss_wildfly	`47.36% <ø> (ø)`
kafka	`64.70% <ø> (ø)`
kubevirt_api	`82.75% <ø> (ø)`
kubevirt_controller	`85.36% <ø> (ø)`
kubevirt_handler	`91.32% <ø> (ø)`
kyverno	`82.27% <ø> (ø)`
lighttpd	`83.64% <ø> (ø)`
linkerd	`84.70% <ø> (ø)`
milvus	`92.30% <ø> (ø)`
nginx_ingress_controller	`98.55% <ø> (ø)`
openldap	`96.33% <ø> (ø)`
openmetrics	`98.05% <ø> (ø)`
pdh_check	`95.65% <ø> (ø)`
php_fpm	`90.45% <ø> (ø)`
postfix	`88.04% <ø> (ø)`
powerdns_recursor	`96.65% <ø> (ø)`
presto	`?`
prometheus	`94.17% <ø> (ø)`
pulsar	`100.00% <ø> (ø)`
quarkus	`100.00% <ø> (ø)`
riak	`99.21% <ø> (ø)`
silverstripe_cms	`76.00% <ø> (ø)`
solr	`56.25% <ø> (ø)`
sonarqube	`98.10% <ø> (ø)`
sonatype_nexus	`76.69% <ø> (ø)`
ssh_check	`91.32% <ø> (ø)`
statsd	`87.36% <ø> (ø)`
strimzi	`89.78% <ø> (ø)`
supervisord	`90.14% <ø> (ø)`
system_swap	`98.30% <ø> (ø)`
tekton	`82.45% <ø> (ø)`
teleport	`98.16% <ø> (ø)`
temporal	`100.00% <ø> (ø)`
teradata	`94.27% <ø> (ø)`
tomcat	`60.41% <ø> (ø)`
twemproxy	`79.45% <ø> (ø)`
voltdb	`96.84% <ø> (ø)`
weaviate	`76.27% <ø> (ø)`
weblogic	`?`
windows_performance_counters	`98.36% <ø> (ø)`
windows_service	`94.53% <ø> (ø)`
wmi_check	`92.91% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

github-actions · 2025-06-05T11:01:49Z