Skip to content

[K9VULN-11268] Handle ignore directives#804

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits intomainfrom
julien/K9VULN-11268
Jan 31, 2026
Merged

[K9VULN-11268] Handle ignore directives#804
gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits intomainfrom
julien/K9VULN-11268

Conversation

@juli1
Copy link
Collaborator

@juli1 juli1 commented Jan 30, 2026

What problem are you trying to solve?

We want to ignore secrets detected on lines where there is a dd-no-secrets directive (comments) on the line above.

What is your solution?

Implement the solution.

@juli1 juli1 requested a review from a team as a code owner January 30, 2026 20:19
@juli1
Copy link
Collaborator Author

juli1 commented Jan 30, 2026

/merge

@gh-worker-devflow-routing-ef8351
Copy link

gh-worker-devflow-routing-ef8351 bot commented Jan 30, 2026

View all feedbacks in Devflow UI.

2026-01-30 20:28:25 UTC ℹ️ Start processing command /merge


2026-01-30 20:28:31 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 16m (p90).


2026-01-30 20:47:02 UTCMergeQueue: The checks failed on this merge request

Tests failed on this commit c87284e:

What to do next?

  • Investigate the failures and when ready, re-add your pull request to the queue!
  • If your PR checks are green, try to rebase/merge. It might be because the CI run is a bit old.
  • Any question, go check the FAQ.

@juli1
Copy link
Collaborator Author

juli1 commented Jan 31, 2026

/merge

@gh-worker-devflow-routing-ef8351
Copy link

gh-worker-devflow-routing-ef8351 bot commented Jan 31, 2026

View all feedbacks in Devflow UI.

2026-01-31 00:05:46 UTC ℹ️ Start processing command /merge


2026-01-31 00:05:50 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 16m (p90).


2026-01-31 00:23:21 UTC ℹ️ MergeQueue: This merge request was merged

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