Visitar URL original
Pre-commit hook · Issue #348 · JuliaTesting/Aqua.jl · GitHub
Skip to content

Pre-commit hook #348

Open
Open

Description

@gdalle

Activity

  1. jonas-schulze commented on Oct 30, 2025

    @jonas-schulze

    I think we should find a proper solution to pre-commit/pre-commit#3497 first. 😕

  2. jonas-schulze commented on Oct 30, 2025

    @jonas-schulze

    For the time being, I would recommend that we pursue the Runic approach with one difference. While runic-pre-commit is versioned independently from Runic, I recommend that the (yet to be created) project aqua-pre-commit has releases matching the one's of Aqua. This way, users can use pre-commit autoupdate on smth like this:

    repos:
      - repo: https://github.com/JuliaTesting/aqua-pre-commit # yet to be created
        rev: v0.8.14
        hooks:
          - id: aqua

    and don't have to maintain additional_dependencies:

    repos:
      - repo: https://github.com/JuliaTesting/aqua-pre-commit
        rev: v1.0.0 # versioned independently
        hooks:
          - id: aqua
            additional_dependencies:
              - Aqua@0.8.14 # most likely not managed by `pre-commit autoupdate`

    To automate the version update, option 1 is that everything is handled by Aqua's CI. That is, whenever a new version of Aqua is released, Aqua's CI should update the [compat] entry in aqua-pre-commit/Project.toml to match Aqua's version exactly, and release a new version of aqua-pre-commit. Option 2 would be that aqua-pre-commit's CI polls the latest release of Aqua.jl on a weekly basis (or so) and updates itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions