Skip to content

adding .pre-commit-config.yaml for opm-tests#1437

Open
GitPaean wants to merge 1 commit into
OPM:masterfrom
GitPaean:pre-commit-for-opm-data
Open

adding .pre-commit-config.yaml for opm-tests#1437
GitPaean wants to merge 1 commit into
OPM:masterfrom
GitPaean:pre-commit-for-opm-data

Conversation

@GitPaean

Copy link
Copy Markdown
Member

limiting the files to be with certain extensions
files: '.(?:DATA|data|INC|inc|param|sh|SCH|sch(?:[1-6])?|sched|md|yaml)$'

The following are all the extensions exisiting in the opm-tests.

Likely ASCII: DATA, data, dat, INC, inc, INCL, GRDECL, grdecl, grid, gridopts, Ecl, ecl, CFG, CMD, dimens, done, eqldims, eqlnum, equil, faults, fipnum, fipzon, gitignore, LGR, LOG, multnum, multregt, ntg, param, perm, permx, poro, prop, prpecl, pvt, pvtnum, regdims, relperm1, relperm2, sch, SCH, sch1..sch6, sched, RFT, PRT, thpres, trans, swatinit, tabdims, txt, vfp, VFP, vfpprod, md, yaml, xml, py, sh.
Likely binary/artifacts to exclude: INIT, UNRST, SMSPEC, UNSMRY, RSSPEC, EGRID, DBG, DEBUG, MSG, RSM, INSPEC, ECLRUN, ECLEND, smry, summary*, images jpg, png, docs pdf, h5, odp.

limiting the files to be with certain extensions
files: '\.(?:DATA|data|INC|inc|param|sh|SCH|sch(?:[1-6])?|sched|md|yaml)$'

The following are all the extensions exisiting in the opm-tests.

Likely ASCII: DATA, data, dat, INC, inc, INCL, GRDECL, grdecl, grid, gridopts, Ecl, ecl, CFG, CMD, dimens, done, eqldims, eqlnum, equil, faults, fipnum, fipzon, gitignore, LGR, LOG, multnum, multregt, ntg, param, perm, permx, poro, prop, prpecl, pvt, pvtnum, regdims, relperm1, relperm2, sch, SCH, sch1..sch6, sched, RFT, PRT, thpres, trans, swatinit, tabdims, txt, vfp, VFP, vfpprod, md, yaml, xml, py, sh.
Likely binary/artifacts to exclude: INIT, UNRST, SMSPEC, UNSMRY, RSSPEC, EGRID, DBG, DEBUG, MSG, RSM, INSPEC, ECLRUN, ECLEND, smry, summary*, images jpg, png, docs pdf, h5, odp.
@GitPaean

GitPaean commented Nov 27, 2025

Copy link
Copy Markdown
Member Author

not totally sure it is something welcomed, just creating for discussion.

currently, it limits only check files with certain extensions that we likely to view and edit, instead of using text to cover all the ASCII text files.

If agreed, we can just pre-commit run --all-files.

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.

1 participant