feedback
feedback copied to clipboard
[Regession] [CLI Uploader] [Windows] --file option not working as in legacy uploader
Describe the bug
--file path/to/reports/**/*.xml in legacy uploader is enough to resolve the globing and pickup all xml files under subfolders of path/to/reports
In Cli this doesn't work and fails to find the files. Workaround there is --dir path/to/reports --file *.xml
Environment (please complete the following information):
- Browser: All
- System: All
CLI version 0.7.5
To Reproduce Steps to reproduce the behavior:
- Place some coverage xml reports in folders under
path/to/reports - Run Cli upload-process passing
--file path/to/reports/**/*.xml - Upload fails because it can't find the xml files
Expected behavior Work as in legacy uploader
@nikosatwork how are you uploading? Using the CLI directly? The Orb? The Action?
Can you post a link to your CI run or at least the relevant Codecov snippet?
Hi @thomasrockhu-codecov - we use CLI directly, in our own CI. I am still testing it, and I think I did not experience any issues with --file, so something else might have gone wrong that time.
So unless there isn't anything similar reported for the uploader, I would say it can close. Thank you.
huh, that is strange. I'm not seeing any other mentions of this kind of issue. I'll close for now, but as always, feel free to reopen and tag me
@thomasrockhu-codecov I am afraid we'll need to reopen, as I managed to get more info.
So this is a problem with the Windows version only.
- When passing just
--file path/to/reports/*.xmlit fails with:
ExitCode: 2
Usage: codecov.exe upload-process [OPTIONS]
Try 'codecov.exe upload-process -h' for help.
Error: Got unexpected extra arguments
- When passing it like
--file *.xml --dir path/to/reportsit is fine & finds the reports, etc
As we are migrating from legacy to cli, and did not want to have to change the various jobs using it, we created a patch in our CI which converts path/to/reports/*.xml into --file *.xml --dir path/to/reports, only on Windows
But ideally Cli should work as Legacy did