Skip to content

Conversation

@bska
Copy link
Member

@bska bska commented Dec 17, 2025

This way, it's easier to process the convertECL output in a script or redirect its output to a file for subsequent viewing.

@bska bska added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Dec 17, 2025
@bska
Copy link
Member Author

bska commented Dec 17, 2025

jenkins build this please

Copy link
Member

@akva2 akva2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This way, it's easier to process the 'convertECL' output in a
script or redirect its output to a file for subsequent viewing.

While here, also add copyright notice and <config.h>.
@bska bska force-pushed the convertecl-no-colour-for-non-tty branch from 1d6fafb to 37ef704 Compare December 17, 2025 15:28
@bska
Copy link
Member Author

bska commented Dec 17, 2025

jenkins build this please

jenkins4opm pushed a commit to jenkins4opm/opm-tests that referenced this pull request Dec 17, 2025
        PR OPM/opm-simulators#6688

Reason: PR OPM/opm-common#4871
        PR OPM/opm-simulators#6688

opm-common     = 5dac62437a1f6c88bf78681fc412ff637a5ab95d
opm-grid       = b0102dad2cf3f43e525dadfff0193dd7818b90e2
opm-simulators = 832f23033dbd072795106fcad39eb1861b012c0f

### Changed Tests ###

  * 0_base_model2
@bska
Copy link
Member Author

bska commented Dec 17, 2025

PR approved and build check is green. I'll merge into master.

@bska bska merged commit ca6e618 into OPM:master Dec 17, 2025
2 checks passed
@bska bska deleted the convertecl-no-colour-for-non-tty branch December 17, 2025 15:49
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.

2 participants