ci: skip DCO check for dependabot and github-actions bots

[Krishan27]

Bots cannot add Signed-off-by lines to their commits, so the DCO check always fails for dependabot PRs and any automated workflow commits.

Adds an if condition to skip the job when github.actor is dependabot[bot] or github-actions[bot]. Human contributor PRs are unaffected.

Fixes: #88 #89 #90 #91 #92

Summary by CodeRabbit

• Chores
  • Optimized CI/CD workflow to prevent unnecessary signature verification failures for automated processes.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 0c7f614a-d0b4-4f07-a193-d04454de9f2e

📥 Commits

Reviewing files that changed from the base of the PR and between c58b9a1 and cf0d817.

📒 Files selected for processing (1)

• .github/workflows/dco.yml

---

📝 Walkthrough

Walkthrough

The DCO workflow gains an if condition that prevents the check from running when github.actor is dependabot[bot] or github-actions[bot], exempting automated bot actors from the commit sign-off requirement.

Changes

DCO Bot Exclusion

Layer / File(s)	Summary
Skip DCO check for bot actors .github/workflows/dco.yml	Adds an if guard on the DCO check job so it does not execute when github.actor equals dependabot[bot] or github-actions[bot].

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A bunny checks commits with care,
But bots need not sign, that's fair!
dependabot hops on through,
github-actions too — it's true,
No DCO block, just smooth sailing there! 🐇✨

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	The PR changes are unrelated to the linked issue #88, which concerns updating astro dependencies, not CI/DCO configuration.	Either link the correct issue for DCO configuration changes or remove the incorrect linked issue #88 and ensure alignment between PR objectives and linked issues.
Out of Scope Changes check	⚠️ Warning	The PR modifies DCO workflow configuration, which is out of scope relative to the linked issue #88 concerning astro dependency updates.	Ensure the PR's changes align with its linked issues, or update the linked issues to reflect the actual scope of this DCO configuration change.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: skipping DCO checks for bot accounts in CI configuration.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/dco-skip-bots

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cloudflare-workers-and-pages]

Deploying flaglint with    Cloudflare Pages

Latest commit:	1da611c
Status:	✅  Deploy successful!
Preview URL:	https://83638940.flaglint.pages.dev
Branch Preview URL:	https://fix-dco-skip-bots.flaglint.pages.dev

View logs