docs: add 'Platform grammar' reference
Metadata
Current evaluation
Merged documentation for the for statement and platform grammar. Approved by three reviewers and passed all CI checks. Includes placeholders for Craft Grammar 2.3.0 features, now enabled following PR 207.
Suggested action: —
No scores available.
Issue body
- [X] Have you followed the guidelines for contributing?
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [X] Have you successfully run `make lint && make test`?
---
Document the `for` statement. I included some bits that we can uncomment when Imagecraft incorporates Craft Grammar 2.3.0, which includes the `any` platform and `for-else` statements.
I also added an example that isn't app-agnostic, but addresses what will be a common use of `for` statements in Imagecraft. If we think this is too far off (or just unwanted here), I can remove it.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged documentation for the for statement and platform grammar. Approved by three reviewers and passed all CI checks. Includes placeholders for Craft Grammar 2.3.0 features, now enabled following PR 207. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged documentation for the platform grammar for statement. Commented-out sections for Craft Grammar 2.3.0 features were enabled after PR 207 merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged documentation for the platform grammar for statement. PR 207 later merged, enabling previously commented Craft Grammar 2.3.0 features like any platform and for-else statements. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2590 docs: add platform naming rules | charmcraft | merged | Merged documentation update adding platform naming rules from craft-application. Approved by two reviewers. Integrated despite non-blocking CI test failures and type checker notifications. | |
| #1130 docs: add platform naming rules | rockcraft | merged | Merged documentation update adding platform naming rules from Craft Application. Approved by three reviewers with all CI checks passing. A noted merge conflict was resolved prior to final merge. | |
| #6092 docs(reference): add platform naming rules | snapcraft | merged | Merged documentation update adding platform naming rules. The change also bumps the craft-application minor release to resolve a lint dependency. Approved by two reviewers and passed CI checks before integration. | |
| #2080 docs: add platforms reference and how-to | charmcraft | merged | Merged documentation updates for Charmcraft platforms, bases, and architectures, covering multi-base support. Approved by three reviewers, passed CI checks, and resolved issue #2009. | |
| #495 feat: add craft-platforms compatibility methods | craft-application | merged | Merged after approval by two reviewers and passing CI checks. Added craft-platforms compatibility methods and deprecation warnings. Documentation breakage was later resolved in PR #498. | |
| #108 feat: add 'for' statement | craft-grammar | merged | Merged. Implemented a for statement in craft-grammar, adding integration tests per reviewer feedback. CI passed and three reviewers approved. Minor null-handling feedback was deferred to future issues. |