Skip to content

Fix Security tab name in community health files - #23

Draft
razo7 wants to merge 1 commit into
mainfrom
fix/security-tab-reference
Draft

Fix Security tab name in community health files#23
razo7 wants to merge 1 commit into
mainfrom
fix/security-tab-reference

Conversation

@razo7

@razo7 razo7 commented Aug 24, 2026

Copy link
Copy Markdown
Member

Why

The GitHub tab is named Security, not "Security and quality". PR#22 fixed
SECURITY.md but missed the same reference in CODE_OF_CONDUCT.md and
CONTRIBUTING.md.

Changes

  • CODE_OF_CONDUCT.md: "Security and quality" → Security
  • CONTRIBUTING.md: same fix

Issues

Follow-up to #22

Test plan

  • Verify the Security tab link works from each file's GitHub rendering

The GitHub tab is named "Security", not "Security and quality".
SECURITY.md was already corrected in PR#22.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c7c44773-85e0-4983-8389-72f5f2f89ce1

📥 Commits

Reviewing files that changed from the base of the PR and between 1ef88b8 and 4486c87.

📒 Files selected for processing (2)
  • CODE_OF_CONDUCT.md
  • CONTRIBUTING.md

Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Updated reporting instructions in the Code of Conduct and contribution guidelines to direct private security reports through the repository’s Security tab.

Walkthrough

The private reporting instructions in two repository documents now reference the Security tab instead of the Security and quality tab.

Changes

Security reporting guidance

Layer / File(s) Summary
Update private reporting instructions
CODE_OF_CONDUCT.md, CONTRIBUTING.md
Both documents now direct private vulnerability reports through the repository’s Security tab.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 4486c

This PR corrects the Security tab name in two community health files without changing runtime behavior or user data; no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: mpryc

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: correcting the Security tab name in community health files.
Description check ✅ Passed The description directly explains the incorrect tab name and identifies the two files updated.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/security-tab-reference

Comment @coderabbitai help to get the list of available commands.

@openshift-ci

openshift-ci Bot commented Aug 24, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: razo7

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

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.

1 participant