Skip to content

Refactor datadog_checks_base and dev #20565

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

Open
wants to merge 1 commit into
base: aarake/format-all-integrations-with-ruff
Choose a base branch
from

Conversation

AAraKKe
Copy link
Contributor

@AAraKKe AAraKKe commented Jun 23, 2025

What does this PR do?

This is a followup PR for the one that refactors the rest of packages using ruff. See #20452

Motivation

I needed to create this to extract it from the rest because the compute matrix step produces zero results if any of these pacakges is included in the modified files.

This should allow all tests to run over the reformatted files for the rest of changes.

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

Copy link
Contributor Author

AAraKKe commented Jun 23, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@AAraKKe AAraKKe added the qa/skip-qa Automatically skip this PR for the next QA label Jun 23, 2025 — with Graphite App
@AAraKKe AAraKKe marked this pull request as ready for review June 23, 2025 16:03
@AAraKKe AAraKKe requested review from a team as code owners June 23, 2025 16:03
@AAraKKe AAraKKe requested review from TCheruy and removed request for a team June 23, 2025 16:03
@AAraKKe AAraKKe force-pushed the aarakke/refactor-datadog-checks-base-and-dev branch from 8175c44 to 18d45b4 Compare June 23, 2025 16:22
@AAraKKe AAraKKe force-pushed the aarake/format-all-integrations-with-ruff branch from 77cc690 to 4491e0f Compare June 23, 2025 16:22
@AAraKKe AAraKKe force-pushed the aarakke/refactor-datadog-checks-base-and-dev branch from 18d45b4 to 8c5f914 Compare June 26, 2025 08:25
@AAraKKe AAraKKe force-pushed the aarake/format-all-integrations-with-ruff branch from 4491e0f to 2e5a151 Compare June 26, 2025 08:25
@AAraKKe AAraKKe force-pushed the aarakke/refactor-datadog-checks-base-and-dev branch from 8c5f914 to b1f6953 Compare June 26, 2025 10:58
@AAraKKe AAraKKe force-pushed the aarake/format-all-integrations-with-ruff branch from 2e5a151 to 3860d64 Compare June 26, 2025 10:58
@AAraKKe AAraKKe force-pushed the aarakke/refactor-datadog-checks-base-and-dev branch from b1f6953 to 703a49e Compare June 26, 2025 11:01
@AAraKKe AAraKKe force-pushed the aarake/format-all-integrations-with-ruff branch from 3860d64 to 6e0f6aa Compare June 26, 2025 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants