meta: add support for hooks
Metadata
Current evaluation
Merged into snapcraft/7.0 branch after reviewer approval and passing CI checks. Adds support for hooks in snapcraft.yaml, modifying four files with 116 additions and 24 deletions while maintaining 100% diff coverage.
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 `./runtests.sh static`?
- [X] Have you successfully run `./runtests.sh tests/unit`?
-----
Support for `hooks` in `snapcraft.yaml`.
CRAFT-808
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into snapcraft/7.0 branch after reviewer approval and passing CI checks. Adds support for hooks in snapcraft.yaml, modifying four files with 116 additions and 24 deletions while maintaining 100% diff coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into snapcraft/7.0, adding hooks support to snapcraft.yaml. The maintainer-authored change increased coverage by 0.05% with 100% diff coverage, resolving CRAFT-808. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Added support for hooks in snapcraft.yaml. Merged into the snapcraft/7.0 branch with 100% test coverage and a 0.05% overall coverage increase. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1026 Add support for hooks. | snapcraft | merged | Merged into master to add hook support to snapcraft. Hooks are now supported via convention by placing executables in the snap/hooks directory or installing them from parts. Approved and passed CI checks. | |
| #1042 Add documentation for hooks. | snapcraft | merged | Merged documentation update for snapcraft hook support, resolving Launchpad bug #1586465. The change adds 34 lines to one file, achieves 100% diff coverage, and was successfully integrated into the main branch. | |
| #3657 meta: support plugs | snapcraft | merged | Merged into snapcraft/7.0 after approval and CI checks. Adds metadata support for plugs, including ContentPlugs. Modifies four files with 117 additions and two deletions, achieving 95.45% diff coverage. | |
| #1586465 Add support for hooks | snapcraft (launchpad) | closed | Closed without implementation. The feature request to add snap.yaml hooks support was abandoned or superseded by subsequent snapd developments. | |
| #3837 fix: hooks: process generated hooks | snapcraft | merged | Merged fix to include generated hooks in snap packages. Ports legacy behavior to snapcraft, writing wrappers to meta/hooks/ while preserving pre-existing hooks. Added spread tests for core18, core20, and core22. | |
| #5463 docs: migrate 'hooks' | snapcraft | merged | 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. | |
| #2850 hooks: enable command-chain | snapcraft | merged | Merged changes enabling command-chain support in hooks regardless of base. The update passes through user-defined command-chain configurations from snapcraft.yaml without actively enabling it. Approved by one reviewer and passed CI checks. | |
| #49 packages: add snap package support | craft-parts | merged | Merged after implementing snap download and installation for stage and build snaps. Approved by two reviewers, passed CI, and added 1,072 lines across five files. Follows snapcraft conventions with future refactoring planned. | |
| #3637 meta: generate basic snap.yaml (CRAFT-801) | snapcraft | merged | Merged into snapcraft/7.0, adding automatic generation of a basic snap.yaml to enable snap packing. Approved by a reviewer, passed CI, and increased test coverage by 1.16%. | |
| #3683 commands: add snap command | snapcraft | merged | Merged into snapcraft/7.0. Implements legacy snap command support for package creation and fixes output parameter passing to managed instances. Approved by reviewer with passing CI checks. |