← Back to issue list

docs: Remove Chisel-related duplicated content

View original Github issue

Metadata

Project
rockcraft
Number
#1267
Type
pull request
State
merged
Author
asanvaq
Labels
—
Created
Updated
Closed

Current evaluation

Merged as commit 888acff0: removed outdated, duplicated Chisel how-to docs from Rockcraft, deferring to Chisel's own docs; explanation page now covers Chisel's relation to Rockcraft, and a flaky custom-slice Spread test was dropped.

Resolution: closed resolved

Reason: The PR was approved by two reviewers and merged into main as commit 888acff0, deleting the Chisel-only guide content (+59/-551 across 14 files) and rewriting 'Install a custom package slice' with a placeholder package instead of a concrete one. Maintainer-requested follow-up from @medubelko (removing the Chisel-related Spread test that caused intermittent CI failures) was applied by the author and pushed before merge.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Chiselling how-to section is outdated. Some of the pages contain Chisel-only content that is already covered in the Chisel docs themselves. As these pages are not related to Rockcraft and exist elsewhere. Installing a custom slice was based on previous steps from the deleted guides. Instead of giving a specific package as an example, I added a placeholder that solves this problem. I also updated the explanation page to focus on how Chisel relates to Rockcraft. --- - [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`. - [ ] I've added or updated any relevant documentation. - [x] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as commit 888acff0: removed outdated, duplicated Chisel how-to docs from Rockcraft, deferring to Chisel's own docs; explanation page now covers Chisel's relation to Rockcraft, and a flaky custom-slice Spread test was dropped.
stealth/space-bunny-alpha — closed resolved Merged in canonical/rockcraft#1267 (888acff0), removing outdated, duplicated Chisel guides, clarifying Chisel’s relationship with Rockcraft, and dropping a flaky custom-slice Spread test.
qwen/qwen3.6-35b-a3b — — Merged documentation update removing outdated Chisel content and duplicated guides. Updated the Chisel-Rockcraft relationship explanation and fixed a custom slice installation placeholder. Addressed feedback by removing a flaky Spread test. Approved and merged despite minor CI warnings.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#419 docs: update / remove duplicate content rockcraft merged Merged #419, removing duplicate Rockcraft Parts and Chisel documentation and resolving #405; merge eb10855c, content commit bfce1f94.
79%
#1222 docs: Remove "Shrink a rock with Chisel" tutorial rockcraft merged Merged PR #1222 (commit cd5c9ca3), removing the duplicate “Shrink a rock with Chisel” tutorial in favor of the existing Chisel-a-rock how-to guide.
78%
#1062 docs: linking Chisel docs to slice definitions ROCKS-2605 rockcraft merged Merged PR #1062 (commit b8dd56c1), replacing GitHub repository links with Chisel documentation links for slice definitions.
77%
#587 docs: add rockcraft chisel notes craft-parts merged Docs PR approved and merged, landing as canonical/rockcraft#98 (commit 3a87e663): added Chisel explanations and a Rockcraft Parts reference, fulfilling canonical/rockcraft#89.
76%
#405 Remove the "Rockcraft Parts" page rockcraft closed Merged PR #419 removed the redundant “Rockcraft Parts” page and Chisel content after Craft Parts documentation became the canonical reference.
76%
#269 docs: create separate chisel documentation rockcraft merged Merged PR #269 via commit 6b988c8a, creating separate Chisel documentation across four files and switching static asset paths to relative links after review.
76%
#89 add docs for Chisel rockcraft closed Documentation request fulfilled: PR #98 ('Chisel documentation') was merged as commit 3a87e663, adding a Reference page explaining what Chisel is. Later reorganised in #269/#283 and duplicated content trimmed in #1267.
76%
#255 docs: add chisel and rock explanations rockcraft closed Closed unmerged after being split into PRs #271 and #283, which added the Chisel and rock explanations in commits 073078a0 and ffdde934.
75%
#679 docs: update chisel how-to guides rockcraft merged Merged #679, updating Chisel how-to documentation with chisel find and chisel-releases integration-test guidance.
75%
#464 Make chisel a build snap rockcraft closed Closed unmerged and superseded by canonical/rockcraft#758, which carried forward removing Chisel packaging after craft-parts gained equivalent support.
74%