Skip to content

Conversation

hakonhagland
Copy link
Contributor

This is an alternative to #6432 that used manual pre-commit hooks.

This is an alternative to OPM#6432 that used manual pre-commit hooks.
@atgeirr
Copy link
Member

atgeirr commented Sep 18, 2025

I like this, looks extensible, simple and non-intrusive.

@atgeirr
Copy link
Member

atgeirr commented Sep 18, 2025

jenkins build this please

@daavid00
Copy link
Member

daavid00 commented Sep 18, 2025

It might be better to have a separate file CONTRIBUTING.md as it is usually done in projects, then this could have more extensive information about formats, maximum length of lines, alphabetical order of include files, etc. Any thoughts?

@hakonhagland
Copy link
Contributor Author

It might be better to have a separate file CONTRIBUTING.md as it is usually done in projects

@daavid00 Good idea! See #6484 for an example.

@atgeirr
Copy link
Member

atgeirr commented Sep 18, 2025

jenkins build this please

@daavid00 daavid00 added this to the Release 2025.10 milestone Sep 24, 2025
@daavid00
Copy link
Member

Is it ok to merge this @atgeirr?

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

Labels

manual:irrelevant This PR is a minor fix and should not appear in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants