Skip to content

MNT: migrate inifix-pre-commit hooks#381

Closed
neutrinoceros wants to merge 126 commits into
idefix-code:masterfrom
neutrinoceros:mnt/migrate-ini-pre-commit-hooks
Closed

MNT: migrate inifix-pre-commit hooks#381
neutrinoceros wants to merge 126 commits into
idefix-code:masterfrom
neutrinoceros:mnt/migrate-ini-pre-commit-hooks

Conversation

@neutrinoceros

Copy link
Copy Markdown
Collaborator

close #380
Notes

  • migrating is essential: neutrinoceros/inifix pre-commit hooks are discontinued beyond v6.1.2
  • this new version is smarter: it will exclude pytest.ini automatically, removing the need to write an "anti-pattern" regexp
  • if, in the future, you need to exclude other .ini files, you can use args: [--extend-exclude=myfile.ini] (new in this version).

vreville and others added 30 commits December 3, 2024 10:57
…lay which slope limiter is used, fixed typo in the braginskii doc
updated ShowConfig functions from braginskii tc and viscosity to disp…
update changelog and version in develop
checking that everything works as expected)
glesur and others added 18 commits January 20, 2026 16:34
* add vector potential indices to pydefix
…de#362)

* fix incorrect subviews when using vector_potential in 2.5D
* test: add -cache option to the test script to accelerate the build

* build: Add option -DIdefix_PROBLEM_DIR to allow using the standard out-of-source build way of cmake

---------

Co-authored-by: Geoffroy Lesur <geoffroy.lesur@univ-grenoble-alpes.fr>
* fix UCT HLLD when using DIMENSIONS<COMPONENTS
* move xdmf test in a dedicated directory

* do not use xdmf in sedov test

* move dump checks in a dedicated directory

* add back sedov and IO tests to github actions

* add back the linear wave test, and fix tolerance

* Hard code pybind11 venv path from cmake
@glesur

glesur commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Can you please target the develop branch?

svalat and others added 2 commits June 4, 2026 15:56
* test: add the "default" keyword for testme.json and port all testme.json to it

* test: update doc to document de "default" behavior

* test: fix missing disabling nonRegressionTest for test/HD/SedovBlastWave in testme.json

* test: enable logging all streams (log, stdout, sterr) in junit file (to prepare following patches for CI)
@neutrinoceros neutrinoceros force-pushed the mnt/migrate-ini-pre-commit-hooks branch from 0e5e4b3 to 6db6cdf Compare June 4, 2026 14:19
@neutrinoceros neutrinoceros deleted the mnt/migrate-ini-pre-commit-hooks branch June 4, 2026 14:21
@neutrinoceros

Copy link
Copy Markdown
Collaborator Author

Sorry I could have sworn this patch only applied to master. I re-opened with the correct base as #383

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants