search

ssl-hep / ServiceX_App

REST Front End to ServiceX
BSD 3-Clause "New" or "Revised" License
3 stars 10 forks source link

preflight removal #156

Closed ivukotic closed 2 years ago

ivukotic commented 2 years ago

We don't need it. This will clean up the code. Partial solution to ServiceX Issue 337

codecov[bot] commented 2 years ago

Codecov Report

Merging #156 (f6f072d) into develop (f5a73fd) will decrease coverage by 0.17%. The diff coverage is 91.66%.

:exclamation: Current head f6f072d differs from pull request most recent head 950c947. Consider uploading reports for the commit 950c947 to get more accurate results

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #156      +/-   ##
===========================================
- Coverage    90.80%   90.63%   -0.18%     
===========================================
  Files           51       50       -1     
  Lines         1458     1442      -16     
  Branches       158      158              
===========================================
- Hits          1324     1307      -17     
  Misses         121      121              
- Partials        13       14       +1
Impacted Files Coverage Δ
servicex/lookup_result_processor.py 100.00% <ø> (ø)
servicex/routes.py 100.00% <ø> (ø)
servicex/resources/transformation/submit.py 95.04% <75.00%> (-0.87%) :arrow_down:
servicex/resources/internal/transform_start.py 100.00% <100.00%> (ø)

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 f5a73fd...950c947. Read the comment docs.