Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Made IPC integration tests run tests that are not run by arrow-rs #278

Merged
merged 2 commits into from Aug 31, 2021

Conversation

jorgecarleitao
Copy link
Owner

@jorgecarleitao jorgecarleitao commented Aug 12, 2021

Not sure patching arrow is a good idea, since the patch may fail. It should probably be a bit more configurable there.

I will leave this here for some time and re-trigger the job to check that the patch continues to apply for some time.

@jorgecarleitao jorgecarleitao added the enhancement An improvement to an existing feature label Aug 12, 2021
@codecov
Copy link

codecov bot commented Aug 12, 2021

Codecov Report

Merging #278 (b188df7) into main (640ec4a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #278   +/-   ##
=======================================
  Coverage   80.88%   80.88%           
=======================================
  Files         326      326           
  Lines       21043    21043           
=======================================
  Hits        17021    17021           
  Misses       4022     4022           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 640ec4a...b188df7. Read the comment docs.

@jorgecarleitao jorgecarleitao added testing PRs that only increase coverage and removed enhancement An improvement to an existing feature labels Aug 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
testing PRs that only increase coverage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant