← Back to issue list

docs: add platform naming rules

View original Github issue

Metadata

Project
rockcraft
Number
#1130
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

Describes the rules for naming platforms, bringing in the document from Craft Application. <!-- Describe your changes --> --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/rockcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [x] I've successfully run `make lint && make test`. - [x] I've added or updated any relevant documentation.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged documentation update introducing platform naming rules from Craft Application. A merge conflict was resolved before final integration.
qwen3.6-35b-a3b-mtp-q6
Staleness: 35
Complexity: 10
Confidence: 85
needs review Documents platform naming rules migrated from Craft Application. Currently blocked by a merge conflict. Awaiting author to resolve conflicts before review and merge proceed.

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.
95%
#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.
93%
#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.
77%
#1012 docs: add platform naming snippet craft-application merged Merged after approval and passing CI. Added platform naming documentation across four files, resolving reviewer feedback on YAML constraints by implementing platform-specific rules for snaps and rocks.
77%
#213 docs: add 'Platform grammar' reference imagecraft merged 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.
76%
#2104 docs: update platforms reference charmcraft merged Merged documentation update to the platforms reference, incorporating feedback from PR #2080. Approved by two reviewers with zero unresolved comments. Changed one file with 16 additions and 15 deletions.
74%
#947 feat!(platforms): prevent use of '/' in platform names craft-application merged Prevents use of '/' in platform names, resolving issue #941 and CRAFT-4876. Merged after approval by two reviewers and passing all CI checks across four modified files.
73%
#706 fix: improve the handling of platforms errors. craft-application merged Merged fix for platform error handling. Approved by two reviewers with all CI checks passing. Resolves issue #690 and CRAFT-4369/4370. Modifies eight files with 98 additions and 239 deletions.
73%
#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.
73%
#6069 docs: rewrite platform docs snapcraft merged Squash-merged rewrite of platform documentation. Dropped core20, added core26, and renamed architectures to platforms. Fixes #6039. Approved by two reviewers with all CI checks passing.
72%