Skip to content

docs: rename org check gate setting to "Fail check at severity" (WEB-1275) - #102

Merged
zeyu2001 merged 1 commit into
mainfrom
devin/web-1275-fail-check-at-severity
Oct 30, 2026
Merged

zeyu2001 merged 1 commit into
mainfrom
devin/web-1275-fail-check-at-severity

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Brings the Check gate section in line with HacktronAI/web#1662, which renames the org setting from "Minimum severity" to Fail check at severity.

  • The steps now tell readers to look for Fail check at severity under Review defaults. They used to point at a "Severity threshold" card, and that card no longer exists.
  • Added a note that the Hacktron check only blocks the merge when it's marked required in source control branch protection.
  • Replaced the old images/severity_threshold.png, which still showed "Severity threshold / Maximum severity", with images/fail_check_at_severity.png. The new screenshot comes from the PR #1662 preview using the e2e test org.

Checked pnpm lint, mintlify broken-links (no broken links) and a local mintlify dev render of the page.

Fail check at severity

Part of WEB-1275

Link to Devin session: https://app.devin.ai/sessions/02e2e57a59b1463eaf2633a1ea8702ad
Open in Devin Desktop: https://app.devin.ai/desktop/session/02e2e57a59b1463eaf2633a1ea8702ad?variant=devin

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@mintlify

mintlify Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
hacktronai 🟢 Ready View Preview Oct 30, 2026, 11:46 AM

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-30T11:47:11.923290Z 3c030d7 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3c030d7ac4

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

## Check gate

Set an org-wide severity threshold that fails a PR or MR check when a finding meets or exceeds it.
Set an org-wide severity that fails the Hacktron check on a PR or MR when a finding meets or exceeds it. To block the merge, make the Hacktron check required in your source control's branch protection.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Provide the correct GitLab merge-blocking setting

For GitLab repositories, this instruction sends readers to branch protection even though Hacktron publishes a GitLab commit status (code-review/findings-feedback.mdx:118), which GitLab represents as an external pipeline job. To make that status block an MR, a Maintainer or Owner must enable Settings → Merge requests → Merge checks → Pipelines must succeed, rather than edit branch protection, as described in the GitLab documentation. Because this section explicitly covers MRs, GitLab customers following the current instruction can leave a failing Hacktron status non-blocking; provide provider-specific directions or link to the correct setup. .agents/skills/writing-docs/SKILL.mdL15-L18

Useful? React with 👍 / 👎.

@zeyu2001
zeyu2001 merged commit 98182ba into main Oct 30, 2026
7 checks passed

This branch was successfully deployed

1 active deployment
staging — 3c030d7a Deployed Oct 30, 2026 by mintlify[bot]
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.

1 participant