Skip to content

Execute pre_commit_script as a complete path#305

Open
ekohl wants to merge 1 commit intovoxpupuli:masterfrom
ekohl:make-pre-commit-script-free
Open

Execute pre_commit_script as a complete path#305
ekohl wants to merge 1 commit intovoxpupuli:masterfrom
ekohl:make-pre-commit-script-free

Conversation

@ekohl
Copy link
Member

@ekohl ekohl commented Jun 18, 2025

Prior to this the pre_commit_script needed to be in contrib, as shipped by the gem. This gives complete freedom on what to run, which gives a lot more freedom.

Currently untested, but @TheMeier recently did a work on cleaning up fixture files. This would allow creating a script that does all the clean ups on every modulesync, and more.

Prior to this the pre_commit_script needed to be in contrib, as shipped
by the gem. This gives complete freedom on what to run, which gives a
lot more freedom.
@ekohl ekohl force-pushed the make-pre-commit-script-free branch from f1b4a81 to 21ce8fe Compare January 13, 2026 16:00
@bastelfreak
Copy link
Member

@TheMeier any chance you can test this?

@TheMeier
Copy link

I will try to look at it. But can't promise anything ATM

@TheMeier TheMeier self-assigned this Jan 22, 2026
@TheMeier
Copy link

Well I had a look into it.

  • The script path MUST be passed as absolute path, this should be documented.
  • It does not work in noop mode, obviuosly
  • If you modify files in the git checkout with the script the changes are not staged by modulesync so it would be the task of the script to do that (fails if not done)

I was actually hoping to use this in a way like https://github.com/lindell/multi-gitter/ to do bulk changes without any other pending modulesync_config triggerd changes.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants