codecov / codecov-exe

.exe report uploader for Codecov https://codecov.io
MIT License
25 stars 21 forks source link

Update accepted commandline arguments to be similiar to codecov-bash #53

Open AdmiringWorm opened 5 years ago

AdmiringWorm commented 5 years ago

Currently there are some difference between codecov-exe and codecov-bash when it comes to the arguments accepted for the commandline.

The accepted arguments should be updated to be similiar for where we can.

AdmiringWorm commented 5 years ago

This have been pushed back to an eventual 2.0 release, as it is expected that this will become a breaking change.

PureKrome commented 4 years ago

One of the important ones (for me) is the -s argument, which I believe is the folder to search for the files, in.

A work around (currently) could be to use powershell to 'find' all the .xml files that the code coverage tool generated (like coverlet) and then call codecov.exe for each result-file, found.

context: coverlet creates a random folder, which it puts the xml file into. And codecov.exe I believe requires an exact file name/path.

AdmiringWorm commented 4 years ago

@PureKrome One of the important ones (for me) is the -s argument, which I believe is the folder to search for the files, in.

A work around (currently) could be to use powershell to 'find' all the .xml files that the code coverage tool generated (like coverlet) and then call codecov.exe for each result-file, found.

context: coverlet creates a random folder, which it puts the xml file into. And codecov.exe I believe requires an exact file name/path.

Codecov.exe have ever since version 1.6.0 allowed to use of globbing patterns instead of using exact file paths. Not familiar with the -s argument that you mentioned, but there is a way to select several files without knowing the full path to the file.

As an example: codecov -f "artifacts/**/*.xml". (Make sure to quote it so codecov expands the globbing and not the shell) Above is just from memory, so the actual format needs to be checked.

reference for available globbing patterns: https://github.com/kthompson/glob/#supported-pattern-expressions

PureKrome commented 4 years ago

👋 @AdmiringWorm

this is the exact info from the official codecov-bash readme.

image

but having the -f <globbing> is really helpful! Maybe throw an example into the ReadMe.md file?

🎉 Cheers for the info! really great stuff! ta!

AdmiringWorm commented 4 years ago

Thank you for that screenshot @PureKrome. I think this would deserve an issue of its own, and should be able to be implemented long before this issue will be completed.

Would you mind opening an issue for that?

I have opened an issue for adding an example of using globbing pattern here: #98