Skip to content

fix(guard): exempt test files from markdown leak detection (Rule 3) - #1701

Merged
Ikalus1988 merged 1 commit into
Ikalus1988:mainfrom
zsxh1990:fix/shape-guard-clean
Sep 19, 2026
Merged

Ikalus1988 merged 1 commit into
Ikalus1988:mainfrom
zsxh1990:fix/shape-guard-clean

Conversation

@zsxh1990

Copy link
Copy Markdown
Contributor

Fixes #1538. Test file exemption for Rule 3. DCO signed, no data churn.

Test files legitimately contain markdown data (triple backticks, ## headers)
which triggers false positives in Rule 3. Add test file exemption.

Fixes Ikalus1988#1538

Signed-off-by: zsxh1990 <445655361@qq.com>
@zsxh1990
zsxh1990 force-pushed the fix/shape-guard-clean branch from 1183aa5 to 4bbf159 Compare September 15, 2026 10:03
@Ikalus1988 Ikalus1988 removed generated-file Contains auto-generated files needs-dco Missing Signed-off-by in commit labels Sep 15, 2026
@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Ikalus1988
Ikalus1988 merged commit aa81323 into Ikalus1988:main Sep 19, 2026
20 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

🎉 Merged — Thank you!

Your contribution has been merged into main.

PR: #1701 — fix(guard): exempt test files from markdown leak detection (Rule 3)

What's next:

  • Your code is now part of MisakaNet's failure-lesson corpus (now 393 lessons)
  • Feel free to pick up another issue labeled good first issue or status: competition
  • Questions? Ask in this thread or open a Discussion

Welcome to the MisakaNet contributor community! 🧠

@github-actions

Copy link
Copy Markdown
Contributor

✅ Merged! Thanks again, @zsxh1990.

fix(guard): exempt test files from markdown leak detection (Rule 3) (+3 lines, 1 files)

Quick question — did any MisakaNet lesson help you this time?
→ Share feedback

No need to reply if nothing comes to mind. ⚡

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