use cli instead of node uploader

[dana-yaish]

Notes to reviewer:

• Currently tokenless upload is not supported. Releasing this PR for users who are not using a codecov token will break their CI
• CLI is now used in 3 platforms: windows, linux and mac. When the node uploader was used with 5 platforms, aarch64, alpine, linux, macos, windows
• when using the file option, the CLI includes other files that it finds including the ones specified with the --file/-f when the node uploader only includes the explicitly specified files. the CLI has an option called --disable-search which doesn't automatically search for coverage files. When used with --flag/-f only those files are included. We can add this option whenever --file/-f is used. Tell my your thoughts on this

[codecov]

Codecov Report

Merging #1068 (142f83f) into main (c9e0f0b) will increase coverage by 0.30%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1068      +/-   ##
==========================================
+ Coverage   95.84%   96.15%   +0.30%     
==========================================
  Files           4        4              
  Lines         337      208     -129     
  Branches      101       59      -42     
==========================================
- Hits          323      200     -123     
+ Misses         14        8       -6     

Flag	Coverage Δ
demo	96.15% <100.00%> (+16.15%)	⬆️
macos-latest	96.15% <100.00%> (+0.30%)	⬆️
script	96.15% <100.00%> (-0.43%)	⬇️
ubuntu-latest	96.15% <100.00%> (+0.30%)	⬆️
version	96.15% <100.00%> (+16.15%)	⬆️
windows-latest	96.15% <100.00%> (+0.30%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
src/buildExec.ts	98.69% <100.00%> (+1.47%)	⬆️
src/helpers.ts	92.50% <100.00%> (+1.32%)	⬆️

[thomasrockhu-codecov]

seems like we should return somethign like

{
    url,
    verbose
}

instead

[dana-yaish]

the reason why we're returning args here, is that it encapsulates building the args with their options names. So if we're returning url and verbose, we would have to wrap each argument with it's option name in index.ts which I don't think is the right thing to do

[thomasrockhu-codecov]

Oh I see, I think we will want to do that eventually to be more clear what args is going to be, but can push off to another PR

[thomasrockhu-codecov]

oh, I know this isn't you, but this is definitely a breaking change. we need aarch64 and alpine builds. cc: @drazisil-codecov

[drazisil-codecov]

@t

oh, I know this isn't you, but this is definitely a breaking change. we need aarch64 and alpine builds. cc: @drazisil-codecov

@thomasrockhu-codecov Did you create a new issue for this?

[thomasrockhu-codecov]

codecov/codecov-cli#266

[thomasrockhu-codecov]

Pending log changes

[thomasrockhu-codecov]

Oh I see, I think we will want to do that eventually to be more clear what args is going to be, but can push off to another PR

[lpsinger]

Will support for tokenless uploads be coming back?