Skip to content

📖 [Docs]: Organization defaults use central PSModule guidance - #80

Merged
Marius Storhaug (MariusStorhaug) merged 5 commits into
mainfrom
route-organization-defaults
Sep 10, 2026
Merged

📖 [Docs]: Organization defaults use central PSModule guidance#80
Marius Storhaug (MariusStorhaug) merged 5 commits into
mainfrom
route-organization-defaults

Conversation

@MariusStorhaug

@MariusStorhaug Marius Storhaug (MariusStorhaug) commented Sep 10, 2026

Copy link
Copy Markdown
Member

Organization-default guidance now keeps this repository's local documentation concise while routing contribution guidance to shared MSX and PSModule sources.

Changed: Scoped documentation routing

README.md identifies this public repository as GitHub's organization-defaults repository. AGENTS.md lists only the local reading order. .github/CONTRIBUTING.md links to the central MSX contribution workflow, MSX Ways of Working, and Process-PSModule documentation.

Adopting this release

No configuration, code, or invocation changes are required beyond selecting this release.

Release impact

  • Effective decision: release:skip (Docs).
  • Semantic effect: No release is produced.
  • Release/base coordinates: No version or release coordinates apply.

Technical details

Consumer change record

The README remains focused on this repository's organization-defaults role. The contribution fallback provides links to the shared contribution guidance without changing inherited community-default files.

Template baseline

The contribution routing follows the established Template-PSModule structure, using central MSX workflow references and Process-PSModule framework guidance.

Maintainer evidence

Reviewed current MSX, Process-PSModule, and Template-PSModule guidance. The MSX msx.no routes are intentional and were verified to render in GitHub Markdown and return HTTP 200; the final diff also passed validation.

Relevant issues (or links)

Related work

  • N/A.

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.

🟡 Changes recommended

The contribution-workflow link currently leads to a landing page that does not expose the promised workflow.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Routes organization-default guidance through the central PSModule repository.

Changes:

  • Clarifies this repository’s organization-default role.
  • Simplifies agent and contributor guidance.
  • Centralizes standards and workflow links.
File summaries
File Description
README.md Documents repository purpose and central guidance.
AGENTS.md Retains only local reading order.
.github/CONTRIBUTING.md Routes contributors to central guidance.
Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 1
  • Review effort level: Balanced

💡 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 Outdated

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.

🟡 Changes recommended

The Process-PSModule documentation link uses a stale path instead of the canonical /docs/ URL.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread .github/CONTRIBUTING.md
@MariusStorhaug
Marius Storhaug (MariusStorhaug) marked this pull request as ready for review September 10, 2026 17:24

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.

🟡 Changes recommended

The routing does not fully match the linked requirements, and a critical repository-visibility invariant was removed.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

Suppressed comments (2)

Previously missed (1) — in code that hasn't changed since the last review.

.github/CONTRIBUTING.md:5

  • Issue #79 specifically asks this fallback to route through PSModule's repository context rather than duplicate cross-organization routes, but these new primary links still send contributors directly to MSX. Replace them with the intended central PSModule contribution-workflow route, or align the issue/PR description if direct MSX routing is intentional.

README.md:3

  • Removing the explicit visibility requirement hides an operational invariant: GitHub only applies these organization-wide default community files while the special .github repository is public. Retain that warning so a future visibility change does not silently disable the fallback.
`PSModule/.github` is GitHub's special public organization-defaults repository.
  • Files reviewed: 3/3 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread README.md

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.

🟡 Changes recommended

The newly added MSX documentation URLs currently lead to 404 pages.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread .github/CONTRIBUTING.md

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

The documentation-only changes are consistent, focused, and satisfy the stated scope.

Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

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.

🟢 Approved

The documentation-only changes are focused, consistent, and contain no unresolved issues.

Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

@MariusStorhaug
Marius Storhaug (MariusStorhaug) merged commit f424237 into main Sep 10, 2026
1 check passed
@MariusStorhaug
Marius Storhaug (MariusStorhaug) deleted the route-organization-defaults branch September 10, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:skip Validate this change without publishing a release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants