HEPCloud / decisionengine

HEPCloud Decision Engine framework
Apache License 2.0
6 stars 26 forks source link

Removed unnecessary comment #559

Closed shreyb closed 2 years ago

shreyb commented 2 years ago

The comment was a TODO to add flag functionality to the webserver operation. This was already done in a previous commit.

codecov[bot] commented 2 years ago

Codecov Report

Merging #559 (cbd07c9) into master (91f7a76) will decrease coverage by 0.03%. The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #559      +/-   ##
==========================================
- Coverage   94.64%   94.61%   -0.04%     
==========================================
  Files          46       46              
  Lines        2934     2934              
  Branches      476      476              
==========================================
- Hits         2777     2776       -1     
  Misses        121      121              
- Partials       36       37       +1     
Flag Coverage Δ
python-3.10 94.51% <0.00%> (ø)
python-3.6 94.35% <0.00%> (-0.04%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../decisionengine/framework/engine/DecisionEngine.py 87.36% <0.00%> (ø)
src/decisionengine/framework/dataspace/maintain.py 99.04% <0.00%> (-0.96%) :arrow_down:

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 91f7a76...cbd07c9. Read the comment docs.