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

@aqilaziz aqilaziz commented May 7, 2026

Copy link
Copy Markdown
Contributor

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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

@mnriem mnriem left a comment

Copy link
Copy Markdown
Collaborator

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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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

mnriem commented May 13, 2026

Copy link
Copy Markdown
Collaborator

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