Skip to content

Fix links to documentation - #28

Merged
samdark merged 1 commit into
masterfrom
fix-docs-links-172
Sep 4, 2026
Merged

Fix links to documentation#28
samdark merged 1 commit into
masterfrom
fix-docs-links-172

Conversation

@samdark

@samdark samdark commented Sep 4, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings September 4, 2026 13:52

Copilot AI 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.

🟢 Approval recommended

Only a minor wording/capitalization nit was found; the link updates themselves look correct and low-risk.

Pull request overview

Updates CONTRIBUTING documentation links to point to the current Yii internal documentation site, improving contributor onboarding by avoiding outdated GitHub blob URLs.

Changes:

  • Replace github.com/yiisoft/docs/blob/master/*.md links with yiisoft.github.io/docs/internals/* links.
  • Adjust list formatting in the prerequisites section to be proper Markdown bullets.
File summaries
File Description
.github/CONTRIBUTING.md Updates internal documentation links to the published docs site and tidies list formatting.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/CONTRIBUTING.md

- [Check roadmap](https://github.com/yiisoft/docs/blob/master/003-roadmap.md).
- [Check roadmap](https://yiisoft.github.io/docs/internals/003-roadmap).
- Check package issues at github. Usually there are some.
@samdark
samdark merged commit b05443b into master Sep 4, 2026
3 checks passed
@samdark
samdark deleted the fix-docs-links-172 branch September 4, 2026 14:43
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.

2 participants