Skip to content

Conversation

TheodorNEngoy
Copy link
Contributor

Flip CIFuzz out of dry‑run now that cli11 exists in OSS‑Fuzz.

@codecov
Copy link

codecov bot commented Oct 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (e4ee3af) to head (5e92b75).
⚠️ Report is 135 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##              main     #1232     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files           17        19      +2     
  Lines         4546      5290    +744     
  Branches         0      1069   +1069     
===========================================
+ Hits          4546      5290    +744     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@TheodorNEngoy
Copy link
Contributor Author

CIFuzz failed because the OSS‑Fuzz project for 'cli11' isn’t merged yet (google/oss-fuzz#14131). Switching back to dry‑run so CI stays green; we’ll flip it on once #14131 lands.

@TheodorNEngoy
Copy link
Contributor Author

Fixed the CIFuzz YAML indentation. Kept dry‑run: true in Build+Run so CI stays green until google/oss-fuzz#14131 merges.

@TheodorNEngoy
Copy link
Contributor Author

CI note: CIFuzz is set to dry‑run so it’s safe to execute. Because this PR comes from a fork, a maintainer needs to approve the workflow run ('Approve workflows to run') per GitHub’s public‑fork approvals. Thanks!

@TheodorNEngoy
Copy link
Contributor Author

CI note: CIFuzz is in dry-run mode (safe). Because this PR comes from a fork, please click Approve and run on the pending workflow checks so CIFuzz can execute.
(Reference: GitHub Docs – Approving workflow runs from public forks)

@TheodorNEngoy
Copy link
Contributor Author

CIFuzz is set to dry-run (safe). Because this is a fork PR, please click Approve and run on the pending workflow checks.

@TheodorNEngoy TheodorNEngoy changed the title ci(cifuzz): enable real fuzzing (remove dry-run) ci(cifuzz): keep dry-run until google/oss-fuzz#14131 merges Oct 18, 2025
@TheodorNEngoy
Copy link
Contributor Author

CIFuzz is in dry‑run per the docs until google/oss-fuzz#14131 is merged, otherwise the job fails. Maintainers: please Approve and run the checks.

@TheodorNEngoy
Copy link
Contributor Author

All checks are green. CIFuzz is intentionally in dry‑run (non‑blocking) and safe to merge. I’ll follow up to enable real fuzzing after google/oss-fuzz#14131 merges.

@phlptp
Copy link
Collaborator

phlptp commented Oct 19, 2025

Is this ready?

@phlptp phlptp merged commit 8225a6a into CLIUtils:main Oct 19, 2025
68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants