-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Refactor datadog_checks_base and dev #20565
Conversation
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.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
8175c44
to
18d45b4
Compare
77cc690
to
4491e0f
Compare
18d45b4
to
8c5f914
Compare
4491e0f
to
2e5a151
Compare
8c5f914
to
b1f6953
Compare
2e5a151
to
3860d64
Compare
b1f6953
to
703a49e
Compare
6e0f6aa
to
43de97b
Compare
703a49e
to
64aae30
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
64aae30
to
cbe4d46
Compare
43de97b
to
298cef3
Compare
cbe4d46
to
d81d092
Compare
298cef3
to
3a655f0
Compare
d81d092
to
25a3f8d
Compare
3a655f0
to
ab1a9e0
Compare
25a3f8d
to
0ab6706
Compare
ab1a9e0
to
390c866
Compare
0ab6706
to
8606be8
Compare
390c866
to
9e8690f
Compare
I am closing this one now that we have changed how ci work and we can test all in stacked prs |
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)
qa/skip-qa
label 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