Skip to content

Fix import of ert_shared -> ert.shared#491

Merged
oysteoh merged 1 commit intoequinor:masterfrom
oysteoh:fix_imports
Aug 16, 2022
Merged

Fix import of ert_shared -> ert.shared#491
oysteoh merged 1 commit intoequinor:masterfrom
oysteoh:fix_imports

Conversation

@oysteoh
Copy link
Copy Markdown
Contributor

@oysteoh oysteoh commented Aug 16, 2022

No description provided.

@berland
Copy link
Copy Markdown
Collaborator

berland commented Aug 16, 2022

Pin ert?

@eivindjahren
Copy link
Copy Markdown
Contributor

Needs "ert >= 2.38.0b5" in setup.py

@oysteoh oysteoh force-pushed the fix_imports branch 2 times, most recently from 7ca8ca2 to b18c61b Compare August 16, 2022 07:04
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #491 (7ca8ca2) into master (b53926c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 7ca8ca2 differs from pull request most recent head b18c61b. Consider uploading reports for the commit b18c61b to get more accurate results

@@           Coverage Diff           @@
##           master     #491   +/-   ##
=======================================
  Coverage   87.56%   87.57%           
=======================================
  Files          49       49           
  Lines        7023     7028    +5     
=======================================
+ Hits         6150     6155    +5     
  Misses        873      873           
Impacted Files Coverage Δ
src/subscript/csv_merge/csv_merge.py 93.79% <100.00%> (+0.04%) ⬆️
src/subscript/csv_stack/csv_stack.py 90.59% <100.00%> (ø)
src/subscript/fmuobs/fmuobs.py 82.18% <100.00%> (+0.10%) ⬆️
src/subscript/hook_implementations/jobs.py 91.42% <100.00%> (+0.51%) ⬆️
src/subscript/casegen_upcars/model.py 98.00% <0.00%> (-0.01%) ⬇️
src/subscript/sunsch/sunsch.py 95.11% <0.00%> (+0.01%) ⬆️
src/subscript/sunsch/time_vector.py 87.41% <0.00%> (+0.16%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@oysteoh oysteoh merged commit cd6d7fc into equinor:master Aug 16, 2022
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.

4 participants