-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Data.frame layers #83
Conversation
Codecov Report
@@ Coverage Diff @@
## master #83 +/- ##
==========================================
+ Coverage 85.71% 87.44% +1.73%
==========================================
Files 5 5
Lines 231 239 +8
==========================================
+ Hits 198 209 +11
+ Misses 33 30 -3
Continue to review full report at Codecov.
|
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
I've completed the merge but... 😭 😭 😭 Tests failing!! But they don't locally 🤷♀️ It looks like they are being triggered by |
I'll fetch the PR! |
The tests pass locally for me too 😭 I'll re-launch the workflows. |
…ll info works" anyways
This is so sad. I'm not sure what's going on. The tests passed yesterday. Since, I merged master into this branch to resolve the merge conflicts. This means the vendor params is also now included. That I believe os the only real change. I did refresh all the fixtures after the merge (f0eff34) but they all worked mine locally. |
Digging into one error (https://github.com/EMODnet/EMODnetWFS/runs/5885060397?check_suite_focus=true#step:6:356) , of failing
I can see a fixture for this response it is referring to ( |
Merge branch 'df' of https://github.com/EMODnet/EMODnetWFS into df
Fix #75
Fix #62
Fix #85
Fix #84
I've done my best to reduce the fixtures size.