Repository navigation
docs: rename org check gate setting to "Fail check at severity" (WEB-1275) - #102
Conversation
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 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. |
There was a problem hiding this comment.
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 👍 / 👎.
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.
images/severity_threshold.png, which still showed "Severity threshold / Maximum severity", withimages/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 localmintlify devrender of the page.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