← Back to issue list

docs(architectures): reword how to section

View original Github issue

Metadata

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

Current evaluation

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.

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`? ----- Use more accurate wording for building for a different architecture. Remove 'cross-compiling' because snapcraft doesn't automatically cross-compile. From @cmatsuoka's suggestion in https://github.com/snapcore/snapcraft/issues/4336#issuecomment-1712592331

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged documentation updates to accurately describe building for different architectures, removing misleading cross-compiling terminology. Changes were successfully integrated into main after passing CI checks.
qwen3.6-35b-a3b-mtp-q6 Merged into main. Documentation was updated to accurately describe building for different architectures, removing the misleading cross-compiling terminology. Changes passed lint and unit tests before integration.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#4349 docs(architectures): add how-to, reference, and explanation snapcraft merged 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.
81%
#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.
79%
#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%
#1345 fix(docs): fix a typo craft-parts merged Merged a documentation fix addressing CRAFT_ARCH_BUILD_ON and CRAFT_ARCH_BUILD_FOR triplet differentiation. Approved by two reviewers with all CI checks passing.
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%
#411 docs: refer to the working directory craft-parts merged Merged documentation update generalizing working directory references to remove Snapcraft-specific details. Approved by two reviewers and passed CI checks.
72%
#2299 docs: fix formatting issues charmcraft merged Merged documentation formatting fixes resolving local build errors. Approved by two reviewers with minimal changes (+3/-1 across two files). Merged despite failing snap-build CI checks.
72%
#1031 docs: add new how-to for parts craft-parts merged Merged documentation update adding a generalized how-to guide for parts. Approved by three reviewers, the change adapts an existing Snapcraft page for broader use, updates the changelog, and adds 68 lines across four files.
71%
#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%
#502 docs: add 'about this documentation' starbase merged Merged after approval by two reviewers and passing all CI checks. Added mandatory 'about this documentation' page, modifying three files (+49/-1) to align with canonical/snapcraft standards.
71%