Skip to content

docs: add markdown alert callouts to genuine gotchas - #23

Merged
zaccesss merged 1 commit into
mainfrom
docs/add-callouts
Aug 5, 2026
Merged

docs: add markdown alert callouts to genuine gotchas#23
zaccesss merged 1 commit into
mainfrom
docs/add-callouts

Conversation

@zaccesss

@zaccesss zaccesss commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Adds GitHub markdown alert callouts to three genuine gotchas found while auditing the docs after #19 and #21. Used sparingly, only where a callout replaces something that was already a real prerequisite, warning or licensing caveat, not decoratively.

  • README.md Local setup: the database schema is not in the repo, which blocks step 3 if skipped. Now an IMPORTANT callout instead of a parenthetical.
  • SECURITY.md: "do not open a public GitHub issue" was bold text, now a WARNING callout since it is an instruction that changes reporter behaviour.
  • NOTICE.md: the campus photography licensing exception is now an IMPORTANT callout instead of bold inline text.

Closes #22

Adds an IMPORTANT callout to the README's Local setup for the
missing schema file, a WARNING callout to SECURITY.md for
reporting vulnerabilities privately and an IMPORTANT callout to
NOTICE.md for the campus photography licensing exception.

Closes #22
@zaccesss zaccesss added documentation Improvements or additions to documentation chore Housekeeping quick-win Small and high value, do in a spare moment autoclose Close this issue when a PR that references it merges labels Aug 5, 2026

@zaccesss zaccesss left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Checked before merging:

  • Callouts render correctly in preview: IMPORTANT in README.md and NOTICE.md, WARNING in SECURITY.md.
  • Only used where a genuine prerequisite, warning or licensing gotcha already existed, nothing decorative added.
  • No wording lost, the surrounding prose still reads correctly with the callout in place.
  • CI (PHP syntax check, CodeQL, Analyze) all pass on this docs-only change.
  • PR shows mergeable=MERGEABLE, no conflicts.

Merging with squash.

@zaccesss
zaccesss merged commit 1bde173 into main Aug 5, 2026
3 checks passed
@zaccesss
zaccesss deleted the docs/add-callouts branch August 5, 2026 12:49
@zaccesss
zaccesss restored the docs/add-callouts branch August 5, 2026 18:40
@zaccesss
zaccesss deleted the docs/add-callouts branch August 5, 2026 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autoclose Close this issue when a PR that references it merges chore Housekeeping documentation Improvements or additions to documentation quick-win Small and high value, do in a spare moment

Development

Successfully merging this pull request may close these issues.

Add markdown alert callouts to genuine gotchas in the docs

1 participant