HEPCloud / decisionengine

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

EL7 doesn't ship with a new enough golang for jsonnetfmt #543

Closed jcpunk closed 2 years ago

jcpunk commented 2 years ago

Turns out my test EL7 box had a newer golang in /usr/local/bin/ from when I was hacking on jsonnet for packaging purposes.

codecov[bot] commented 2 years ago

Codecov Report

Merging #543 (cda5511) into master (e95071f) will increase coverage by 0.16%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #543      +/-   ##
==========================================
+ Coverage   83.83%   83.99%   +0.16%     
==========================================
  Files          48       48              
  Lines        3099     3099              
  Branches      483      483              
==========================================
+ Hits         2598     2603       +5     
+ Misses        463      461       -2     
+ Partials       38       35       -3     
Flag Coverage Δ
python-3.10 83.44% <ø> (ø)
python-3.6 83.29% <ø> (+0.12%) :arrow_up:
python-pypy-3.7 83.55% <ø> (+0.19%) :arrow_up:

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

Impacted Files Coverage Δ
.../decisionengine/framework/engine/DecisionEngine.py 90.48% <0.00%> (+0.48%) :arrow_up:
src/decisionengine/framework/dataspace/maintain.py 100.00% <0.00%> (+0.95%) :arrow_up:
src/decisionengine/framework/engine/Workers.py 90.27% <0.00%> (+2.77%) :arrow_up:

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 e95071f...cda5511. Read the comment docs.

jcpunk commented 2 years ago

Yep, I was hopeful to get the autoformatting all in place but alas....