Skip to content

feat(ci): run DetectorChecksum test #850

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 4 commits into
base: main
Choose a base branch
from
Open

Conversation

wdconinc
Copy link
Contributor

@wdconinc wdconinc commented Apr 26, 2025

Briefly, what does this PR introduce?

This PR adds a test to make sure the DetectorChecksum works. Some detectors manage to do things that break it...

Needs:

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue: some detectors manage to break checksum, so make sure that is surfaced)
  • Documentation update
  • Other: __

Please check if this PR fulfills the following:

  • Tests for the changes have been added
  • Documentation has been added / updated
  • Changes have been communicated to collaborators

Does this PR introduce breaking changes? What changes might users need to make to their code?

No.

Does this PR change default behavior?

No.

@github-actions github-actions bot added the topic: infrastructure Regarding build system, CI, CD label Apr 26, 2025
@wdconinc wdconinc force-pushed the detector-checksum branch from 2399531 to 55bc4dc Compare April 28, 2025 18:42
@wdconinc wdconinc force-pushed the detector-checksum branch from 55bc4dc to 93ab9d8 Compare April 28, 2025 21:19
@wdconinc wdconinc requested a review from veprbl April 28, 2025 22:11
@wdconinc wdconinc enabled auto-merge (squash) April 28, 2025 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Regarding build system, CI, CD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant