Do not run CI for all Go versions and use caching#606
Do not run CI for all Go versions and use caching#606rdimitrov merged 1 commit intotheupdateframework:masterfrom
Conversation
Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #606 +/- ##
=======================================
Coverage 70.18% 70.18%
=======================================
Files 10 10
Lines 2123 2123
=======================================
Hits 1490 1490
Misses 517 517
Partials 116 116
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
kommendorkapten
left a comment
There was a problem hiding this comment.
Looks good, is the windows tests expected to fail?
@kommendorkapten - Yes, they were disabled before, but once we enabled them we saw there're a few issues when targets use the prefixed format we use in the tuf-demo repo. I've opened an issue, but from what I see it's probably caused by the wrong handling of back/forward slashes when we are translating a target name to a local path name when we create or look for the target file locally. I don't have a windows machine to properly debug and fix this, so I thought it's better to leave the tests enabled and failing until we get some help on this. Also this way we are open in sharing windows support is not working properly yet. |
The following PR: