Skip to content

feature/align documentation standards#177

Open
JohannesRudolph wants to merge 3 commits into
mainfrom
feature/align-documentation-standards
Open

feature/align documentation standards#177
JohannesRudolph wants to merge 3 commits into
mainfrom
feature/align-documentation-standards

Conversation

@JohannesRudolph
Copy link
Copy Markdown
Member

  • docs(scorecard): add no-documentation_md rule and scorecard criterion
  • refactor(azure): remove legacy documentation_md output from all backplane modules

Add documentation_md anti-pattern rule to AGENTS.md:
- Document that documentation_md output in backplane/ is a legacy pattern
- Specify user-facing docs belong in BBD readme field
- Specify platform-engineer docs belong in backplane/README.md
- Add to checklist for new modules

Add 'no_documentation_md_output' scorecard check (integration category):
- Checks backplane/ TF files for legacy documentation_md output
- Reports violation with actionable detail message
…lane modules

The documentation_md output pattern is deprecated. Documentation is now split:
- User-facing content belongs in the BBD readme field (meshstack_integration.tf)
- Platform-engineer-facing content belongs in backplane/README.md

Changes per module:
- aks: delete documentation.tf, remove output row from README
- azure-virtual-machine: delete documentation.tf, remove output row from README
- container-registry: delete documentation.tf, remove output row from README
- github-actions-terraform-setup: delete documentation.tf, remove output row from README
- key-vault: delete documentation.tf, remove output row from README
- postgresql: delete documentation.tf, fix README intro (was incorrectly titled Key Vault)
- service-principal: delete documentation.tf, create backplane/README.md (was missing)
- spoke-network: delete documentation.tf, remove output row from README
@aws-amplify-eu-central-1
Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-177.d1o16zfeoh2slu.amplifyapp.com

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.

1 participant