← Back to issue list

docs: migrate 'hooks'

View original Github issue

Metadata

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

Current evaluation

Merged the snap hooks documentation migration from snapcraft.io to Read the Docs in commit 6f4c66e59.

Resolution: closed resolved

Reason: PR #5463 was approved with all CI checks passing and merged. Commit 6f4c66e59 adds the migrated hooks reference at docs/reference/hooks.rst.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/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 `make test`? --- Migrate https://snapcraft.io/docs/supported-snap-hooks and https://snapcraft.io/docs/snapcraft-hook-support to [RTD](https://canonical-snapcraft--5463.com.readthedocs.build/en/5463/reference/hooks/). (SNAPCRAFT-395)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged the snap hooks documentation migration from snapcraft.io to Read the Docs in commit 6f4c66e59.
stealth/space-bunny-alpha — closed resolved Merged PR #5463, moving hooks documentation from snapcraft.io to Read the Docs in commit 6f4c66e59.
stealth/space-bunny-alpha — closed resolved Merged PR #5463 (SNAPCRAFT-395), migrating hooks documentation to Read the Docs and updating three files (+105/-1).
qwen/qwen3.6-35b-a3b — — Merged documentation migration for 'hooks' to ReadTheDocs. Approved by two reviewers with all CI checks passing. Updated three files (+105/-1 lines) to align with SNAPCRAFT-395.
qwen3.6-35b-a3b-mtp-q6 — — Merged documentation updates migrating the 'hooks' reference pages from snapcraft.io to ReadTheDocs. The changes passed lint and test checks, resolving SNAPCRAFT-395.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5466 docs: migrate 'snapcraft.yaml' snapcraft merged Merged the snapcraft.yaml documentation migration into the Read the Docs project-file reference via commit 2670e8278 (#5466).
82%
#1042 Add documentation for hooks. snapcraft merged Merged as commit 10c58b652, adding hook usage documentation and completing the Snapcraft portion of Launchpad bug #1586465.
77%
#3659 meta: add support for hooks snapcraft merged Merged snapcraft#3659 via commit 00eed7dc9 (merge 795294fd5) into snapcraft/7.0, adding hooks support in snapcraft.yaml for CRAFT-808.
74%
#5409 docs: migrate 'migrate between bases' snapcraft merged Merged in snapcraft#5409 via commit 30aeaf036, migrating and restructuring the documentation for changing snap bases.
73%
#5373 docs: migrate 'debug a snap' snapcraft merged Merged the “debug a snap” documentation migration, updating five files (+206/−37); landed as commit 2dfb26407.
73%
#5410 docs: migrate 'adding snap configuration' snapcraft merged Migrated the "adding snap configuration" guide into the current docs structure and merged it in commit 7b1784086 via #5410.
72%
#5462 docs: migrate 'Snapcraft advanced grammar' snapcraft merged Merged the advanced grammar documentation migration in commit b7a56461b; follow-up #5468 restored the Read the Docs build in 0157e5b15.
72%
#1026 Add support for hooks. snapcraft merged PR #1026 merged as commit 01880ee8d ('core: add support for hooks'), adding hook support: executables in snap/hooks/ are copied into the snap, and parts can install hooks into snap/hooks/. Docs followed in #1042 (10c58b652).
71%
#3837 fix: hooks: process generated hooks snapcraft merged Merged as commit c77e3a07a (CRAFT-1159): core22 now includes lifecycle-generated hooks in the snap via meta/hooks wrappers, with snap/hooks taking priority; adds spread tests for core18/20/22.
70%
#1707782 Hook documentation unclear snapcraft (launchpad) closed Closed as invalid after a maintainer clarified the snapd-versus-Snapcraft documentation split and linked the hook-building guide; no cross-link change was recorded.
70%