green-coding-solutions / eco-ci-energy-estimation

Eco CI Energy estimation for Github Actions Runner VMs
MIT License
48 stars 10 forks source link

remove doubled cache instruction #44

Closed shuryoka closed 8 months ago

shuryoka commented 8 months ago

yaml file parsers are marking the cache instruction as doubled - this commit removes the second ocurrance as it is an exact copy of the first one

ribalba commented 8 months ago

@dan-mm could you please take a look. I think we needed the caching for venv?

dan-mm commented 8 months ago

@ribalba - the caching is still there, what was removed was an unnecessary duplicate

ArneTR commented 8 months ago

LGTM. Just talked to Didi, was just a visual oversight from him. Can be merged