Skip to content

Fix constitution reference in README#2491

Merged
mnriem merged 2 commits into
github:mainfrom
aqilaziz:docs-fix-constitution-reference
May 13, 2026
Merged

Fix constitution reference in README#2491
mnriem merged 2 commits into
github:mainfrom
aqilaziz:docs-fix-constitution-reference

Conversation

@aqilaziz
Copy link
Copy Markdown
Contributor

@aqilaziz aqilaziz commented May 7, 2026

Summary

  • replace a broken README link to base/memory/constitution.md with the generated .specify/memory/constitution.md path described elsewhere in the guide

Verification

  • checked local links/assets in README.md
  • git diff --check

@aqilaziz aqilaziz requested a review from mnriem as a code owner May 7, 2026 23:17
@mnriem mnriem requested a review from Copilot May 8, 2026 21:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the README guidance around Claude Code plan validation by correcting a broken reference to the project constitution, aligning it with the generated .specify/memory/constitution.md location used elsewhere in the guide.

Changes:

  • Replaced the outdated base/memory/constitution.md reference with the .specify/memory/constitution.md path in the Step 5 note.
Show a summary per file
File Description
README.md Fixes the constitution reference in the plan-validation guidance to point at the generated .specify memory path.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

Comment thread README.md Outdated
Copy link
Copy Markdown
Collaborator

@mnriem mnriem left a comment

Choose a reason for hiding this comment

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

Please address Copilot feedback

@aqilaziz
Copy link
Copy Markdown
Contributor Author

Addressed the Copilot wording feedback in 1ffe125 by changing the sentence to refer to the constitution in .specify/memory/constitution.md.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new

@aqilaziz
Copy link
Copy Markdown
Contributor Author

Follow-up status: the Copilot re-review on 1ffe125 reports 0 new comments, and all checks are passing. The earlier wording feedback has been addressed; ready for maintainer re-review when convenient.

@mnriem mnriem self-requested a review May 13, 2026 12:41
@mnriem mnriem merged commit 0aae1ec into github:main May 13, 2026
15 checks passed
@mnriem
Copy link
Copy Markdown
Collaborator

mnriem commented May 13, 2026

Thank you!

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