New plugin: gulp
Metadata
Current evaluation
Merged to master as commit 00079ee32, adding the gulp plugin (snapcraft/plugins/gulp.py), a gulp simple-project integration test, and updated plugin-list coverage, closing LP bug #1575880.
Resolution: closed resolved
Reason: PR #563 was merged into snapcraft master as commit 00079ee32 ("New plugin: gulp (#563)"), landing the new gulp plugin and its integration test. Maintainer come-maiz gave +1 after the plugin-list test and autopkgtests passed, and the work shipped in Snapcraft 2.12 (LP #1575880).
Evaluated by: mr-cal-bot
Issue body
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged to master as commit 00079ee32, adding the gulp plugin (snapcraft/plugins/gulp.py), a gulp simple-project integration test, and updated plugin-list coverage, closing LP bug #1575880. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged into master, adding the gulp plugin, a simple-project integration test, and updated plugin-list coverage after maintainer approval. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged a new Gulp plugin in canonical/snapcraft#563, adding gulp support and integration coverage. Test output ordering and copyright updates were fixed; the proxy-related autopkgtest failure was unrelated. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a new Gulp plugin. Initial CI failure stemmed from test output ordering, resolved by updating expected results. Reviewer approved after verifying unrelated autopkgtest flake and confirming copyright updates. Successfully integrated into master. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Added a new gulp plugin. After addressing review feedback regarding a missing integration test and copyright year formatting, and confirming unrelated test failures, the changes were approved and merged with master. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#534
(confidence 62%)
Preceding commit e6035c060 wrapped plugin list output content, which is why the plugin-list integration test in this PR needed updating to include gulp.
-
Related To:
snapcraft#782
(confidence 40%)
Later follow-up commit e68413299 "gulp plugin: fix the npm install prefix" indicates the initial gulp plugin had a follow-up defect fix.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1575880 nodejs plugin needs to be able to build using gulpfiles | snapcraft (launchpad) | closed | Resolved by merged PR #563 (commit 00079ee32, "New plugin: gulp"), which added a standalone gulp plugin instead of extending nodejs. Shipped in snapcraft 2.12 to xenial; verified by ~elopio using integration_tests/snaps/simple-gulp. | |
| #1650603 Update gulp plugin to use get_build_properties and get_pull_properties | snapcraft (launchpad) | closed | Merged snapcore/snapcraft#998 as acf95e174, updating gulp to use get_build_properties() and get_pull_properties() and adding unit and schema tests. | |
| #998 plugins: update gulp plugin to use get_build_properties() and get_pull_properties() | snapcraft | merged | Merged as acf95e174, updating the Gulp plugin to use build/pull property hooks and adding schema and unit tests for Launchpad bug #1650603. |