Skip to content

jenkins: fix update_data with existing PR in opm-tests#4881

Merged
bska merged 3 commits into
OPM:masterfrom
akva2:fix_opm_tests_abs_rev_existing_pr
Jan 6, 2026
Merged

jenkins: fix update_data with existing PR in opm-tests#4881
bska merged 3 commits into
OPM:masterfrom
akva2:fix_opm_tests_abs_rev_existing_pr

Conversation

@akva2
Copy link
Copy Markdown
Member

@akva2 akva2 commented Jan 6, 2026

And some fixes for non-ci tooling.

akva2 added 3 commits January 6, 2026 09:06
…n parsed

this fixes data update PRs with a pre-existing PR in opm-tests.
the data update PR will now include the pre-existing PR as it is
supposed to
not exercised on jenkins but useful for other tooling
@akva2 akva2 added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Jan 6, 2026
@akva2
Copy link
Copy Markdown
Member Author

akva2 commented Jan 6, 2026

jenkins build this please

jenkins4opm pushed a commit to jenkins4opm/opm-tests that referenced this pull request Jan 6, 2026
        PR OPM#1454

Reason: PR OPM/opm-common#4881
        PR OPM#1454

opm-common     = fa67624c72f93ea270e4739571830000c3034d6e
opm-grid       = 90444976a9cabb65c88e0718b6e3527b263717cc
opm-simulators = f884b31c383ef06c03a6f9e6445fcaa9117182f3

### Changed Tests ###

  * spe1_float
  * spe1case1
@akva2
Copy link
Copy Markdown
Member Author

akva2 commented Jan 6, 2026

and confirmed working.

Copy link
Copy Markdown
Member

@bska bska left a comment

Choose a reason for hiding this comment

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

Much appreciated!

I'll merge into master.

@bska bska merged commit abaecbf into OPM:master Jan 6, 2026
3 checks passed
@akva2 akva2 deleted the fix_opm_tests_abs_rev_existing_pr branch January 6, 2026 10:31
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