← Back to issue list

ci: add pre-commit config

View original Github issue

Metadata

Project
craft-providers
Number
#292
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #292 as commit 638a1c9, adding Starbase's pre-commit configuration after #291 to advance the tooling sync tracked by #287.

Resolution: closed resolved

Reason: PR #292 was approved and merged, with commit 638a1c9 adding the pre-commit configuration.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- Add `pre-commit` from starbase. This builds on top of https://github.com/canonical/craft-providers/pull/291, so only the last commit needs review here. See https://github.com/canonical/craft-providers/issues/287. (CRAFT-1726)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #292 as commit 638a1c9, adding Starbase's pre-commit configuration after #291 to advance the tooling sync tracked by #287.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR adding pre-commit configuration from starbase to enforce code formatting and linting, resolving issue #287.

Update history

Date Change
created

Related work

  • Related To: canonical/craft-providers#291 (confidence 100%)

    PR #292 explicitly builds on the preceding tooling and lint changes merged in #291.

  • Related To: canonical/craft-providers#287 (confidence 100%)

    Issue #287 tracked the broader synchronization of craft-providers tooling with Starbase.

Related issues

Issue Project State Summary Similarity
#286 ci: sync configs with starbase craft-providers merged Merged PR #286 as commit 73c340c, syncing .gitignore, .editorconfig, and yamllint.yaml with Starbase as the first configuration synchronization.
79%
#68 tools: add pre-commit starbase merged Merged as commit 7a22e93, adding pre-commit tooling and configuration across three files.
78%
#405 style: add pre-commit to auto-formatting starbase merged Merged PR #405 added pre-commit auto-formatting in commit 18e50d3, updating two files and passing all CI checks.
71%
#144 build(deps): remove yamllint craft-platforms merged Merged PR #144 removed yamllint and updated the pre-commit configuration from Starbase in commit 35b75fc.
71%
#11 tools: add yamllint starbase merged Merged PR canonical/starbase#11 as commit 5a0d09f, adding optional yamllint tooling and configuration across three files.
70%