-
Notifications
You must be signed in to change notification settings - Fork 5.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update local cli executor to use same filename strategy as docker #1981
Conversation
@sonichi can you run oai tests? |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1981 +/- ##
===========================================
+ Coverage 37.38% 47.98% +10.59%
===========================================
Files 64 65 +1
Lines 6879 6910 +31
Branches 1518 1647 +129
===========================================
+ Hits 2572 3316 +744
+ Misses 4085 3317 -768
- Partials 222 277 +55
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…crosoft#1981) * consistent file saving across cli executors * test fixes * feedback * make path * formatting * run timeout test on windows --------- Co-authored-by: Chi Wang <[email protected]>
Replaces #1954
Why are these changes needed?
Related issue number
Checks