← Back to issue list

docs(architectures): add how-to, reference, and explanation

View original Github issue

Metadata

Project
snapcraft
Number
#4349
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged documentation updates adding how-to, reference, and explanation guides for architectures. Resolved issue #4336 after three approvals and passing CI. Minor style suggestions and a negligible coverage drop were noted but did not block the merge.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint`? - [x] Have you successfully run `pytest tests/unit`? ----- View docs [here](https://canonical-snapcraft--4349.com.readthedocs.build/en/4349/) Even though this is a draft, feedback is welcome anytime. cc @rpjday Fixes #4336 (CRAFT-2009)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged documentation updates adding how-to, reference, and explanation guides for architectures. Resolved issue #4336 after three approvals and passing CI. Minor style suggestions and a negligible coverage drop were noted but did not block the merge.
qwen3.6-35b-a3b-mtp-q6 Merged into main, adding how-to, reference, and explanation documentation for architectures. Resolves #4336. Minor style feedback and a negligible coverage decrease were noted but did not block the merge.
qwen3.6-35b-a3b-mtp-q6 Merged documentation updates adding how-to, reference, and explanation guides for architectures. Resolves issue #4336. Minor style feedback was addressed. Changes are now in the main branch.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#4362 docs(architectures): reword how to section snapcraft merged Merged a documentation update that reworded the architecture section to remove the term cross-compiling, clarifying snapcraft does not auto-cross-compile. Approved by two reviewers, passed CI, and merged.
81%
#5685 docs: fix missing link to supported architectures snapcraft merged Merged a documentation fix adding a missing link to supported architectures. The single-file change was approved by two reviewers and merged despite one failing CI check.
75%
#4904 docs: add explanation, howto, and reference for components snapcraft merged Merged after approval and passing CI checks. Added explanation, how-to, and reference documentation for components, resolving issue #4857. Unrelated test failures were noted but did not block the merge.
74%
#5199 docs: fix formatting of Architectures snapcraft closed Closed without merge as a training demonstration. Documentation formatting changes were never reviewed or merged due to the pull request's experimental purpose.
73%
#4436 docs(architectures): remove shorthand notation for core22 architectures snapcraft merged Merged documentation update removing shorthand architecture notation for core22 snaps. The change discourages the notation as Launchpad lacks support, avoiding disruptive code removal. Approved by two reviewers and passed CI.
73%
#5526 fix(docs): Fix missing code block snapcraft merged Merged an external contributor's fix adding a missing code block to the architecture-independent documentation. Approved by two reviewers, passed CI, and integrated.
71%
#467 docs: add a how-to for overriding builds craft-parts merged Merged documentation update adding a how-to guide for overriding builds. Approved by one reviewer, passed CI checks, and added 83 lines across three files.
71%
#4267 docs: add command reference snapcraft merged Merged after two approvals and passing CI. Added command reference documentation across 10 files. Required merging prerequisite PR #4272 to fix ReadTheDocs builds before updating the branch and finalizing the merge.
70%
#126 Docs improvements rockcraft merged Merged documentation improvements after reviewer approval. CLA and CI checks were resolved by rebasing from main, completing the merge.
70%
#604 docs: update inspectors reference fetch-service merged Merged documentation update adding references for undocumented inspectors, reconciling docs with actual behavior, and normalizing style. Approved by one reviewer, passed CI checks, and modified 38 files.
70%