Skip to content

fix: add a fallback for accounts that show "Code security" in settings - #75

Open
mamezou wants to merge 1 commit into
skills:mainfrom
mamezou:fix/code-security-settings-fallback
Open

mamezou wants to merge 1 commit into
skills:mainfrom
mamezou:fix/code-security-settings-fallback

Conversation

@mamezou

@mamezou mamezou commented Aug 25, 2026

Copy link
Copy Markdown

Summary

Step 1 tells learners to select Advanced Security in the repository settings. On some accounts the entry is named Code security instead, and learners who cannot find Advanced Security stop before enabling code scanning.

Observed on 2026-08-25 in a training class: most learners, who created their GitHub accounts that day, saw Code security, while the instructor account, created in 2018, saw Advanced Security. What decides the difference is not confirmed, so this pull request keeps Advanced Security first and adds the other name as a fallback rather than replacing it.

The wording follows the style guide's guidance for elements that are not always visible: "Click Security. If Security is not visible, click ⋮ to expand the repository menu."

Related: #71 reports that the Advanced Security page does not always contain the expected sections.

Changes

  • .github/steps/1-step.md: add "If Advanced Security is not shown, select Code security." to the settings navigation step.

Task list

  • For workflow changes, I have verified the Actions workflows function as expected. (No workflow changes in this pull request.)
  • For content changes, I have reviewed the style guide.

@krisiroy12 krisiroy12 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.

The documentation update is clear and helpful. Adding the Code security option as an alternative makes the instructions easier to follow when Advanced Security is not available.

@PrakashGanam

Copy link
Copy Markdown

Verified

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.

3 participants