Skip to content

docs: update copilot instructions for scadm export-png and ADR convention#16

Merged
kellervater merged 2 commits into
mainfrom
feat/export-png-docs
May 9, 2026
Merged

docs: update copilot instructions for scadm export-png and ADR convention#16
kellervater merged 2 commits into
mainfrom
feat/export-png-docs

Conversation

@kellervater
Copy link
Copy Markdown
Member

📦 What

  • ✅ Update export-png references in openscad and markdown instructions
  • ✅ Change ADR naming convention to kebab-case-title.md (no numeric prefixes)

💡 Why

Keeps copilot instructions in sync across all kellerlabs repos after scadm export-png migration.

🔧 How

Doc-only changes. No functional impact.

📋 Companion PR: kellerlabs/homeracker feat/export-png

…tion

- Update export-png references in openscad and markdown instructions
- Change ADR naming convention to kebab-case (no numeric prefixes)
Copilot AI review requested due to automatic review settings May 9, 2026 07:42
Copy link
Copy Markdown

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 GitHub Copilot and documentation instruction markdown to reflect the scadm export-png workflow and a new (non-numeric) decision-record filename convention.

Changes:

  • Replace cmd/export/export-png.sh / export-png.sh references with scadm export-png in OpenSCAD + Markdown instruction docs.
  • Update the Copilot instructions’ decision-record naming guidance to kebab-case-title.md (no numeric prefixes).
  • Adjust the referenced decision-doc link text/target in Copilot instructions.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
.github/instructions/openscad.instructions.md Updates preview PNG generation guidance to use scadm export-png.
.github/instructions/markdown.instructions.md Updates the model README template to reference scadm export-png for preview generation.
.github/copilot-instructions.md Updates export-png tooling reference and changes the decision-record naming convention + linked decision doc reference.
Comments suppressed due to low confidence (1)

.github/instructions/markdown.instructions.md:75

  • The Catalog example still links previews as parts/part_name.png, but elsewhere these instructions state previews live under a renders/ subfolder (and the Assets Policy references **/renders/*.png). Consider updating the example path to match the documented output location to avoid confusing readers.
## 📸 Catalog

Preview images for each part. Generate with `scadm export-png`:

| Part | Preview |
|------|---------|
| Part Name | ![Part Name](parts/part_name.png) |

Comment thread .github/copilot-instructions.md Outdated
Comment thread .github/copilot-instructions.md
@kellervater kellervater merged commit 9557df4 into main May 9, 2026
3 checks passed
@kellervater kellervater deleted the feat/export-png-docs branch May 9, 2026 08:13
kellerlab-release-please Bot added a commit that referenced this pull request May 11, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.3.2](pre-commit-hooks-v0.3.1...pre-commit-hooks-v0.3.2)
(2026-05-09)


### 📚 Documentation

* update copilot instructions for scadm export-png and ADR convention
([#16](#16))
([9557df4](9557df4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: kellerlab-release-please[bot] <246533846+kellerlab-release-please[bot]@users.noreply.github.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.

2 participants