Skip to content
This repository was archived by the owner on May 21, 2025. It is now read-only.

Fix command line quoting for Windows #169

Merged
thomasrockhu merged 3 commits into from
May 26, 2020
Merged

Fix command line quoting for Windows #169

thomasrockhu merged 3 commits into from
May 26, 2020

Conversation

@ofek
Copy link
Contributor

@ofek ofek commented Oct 25, 2018

cc @stevepeak Single quotes do not work in cmd, only powershell:

https://ci.appveyor.com/project/Datadog/integrations-core/builds/19797599#L705

capture

Without git ls-files results, path combining does not work cross-platform

@ofek ofek mentioned this pull request Oct 25, 2018
Merged
@ofek
Copy link
Contributor Author

ofek commented Jul 13, 2019

cc @thomasrockhu 😄

@codecov
Copy link

codecov bot commented Sep 12, 2019 •
edited
Loading

Codecov Report

Merging #169 into master will not change coverage by %.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #169   +/-   ##
=====================================
  Coverage       0%     0%           
=====================================
  Files           1      1           
  Lines           1      1           
=====================================
  Misses          1      1

ofek added a commit to ofek/codecov-python that referenced this pull request Sep 12, 2019
@ofek
fix quoting
3bf567a 
codecov#169
@ofek
Copy link
Contributor Author

ofek commented Sep 12, 2019

@thomasrockhu Can we merge?

@nmoinvaz
Copy link
Contributor

nmoinvaz commented Oct 9, 2019 •
edited
Loading

Please merge this change. It works for me in Windows environment.

This was referenced Mar 1, 2020
Closed
Closed
@ofek
Copy link
Contributor Author

ofek commented Mar 25, 2020

cc @drazisil

@drazisil
Copy link
Contributor

drazisil commented Mar 25, 2020

@ofek

I'm out sick for now, i will take a look when i get back, thanks for the ping.

@ofek
Copy link
Contributor Author

ofek commented Mar 25, 2020

No worries, get well!

@drazisil drazisil self-assigned this Mar 25, 2020
@drazisil drazisil self-requested a review March 25, 2020 21:08
@ofek
Copy link
Contributor Author

ofek commented May 26, 2020

Any update?

@thomasrockhu thomasrockhu self-requested a review May 26, 2020 22:01
@thomasrockhu
Copy link
Contributor

thomasrockhu commented May 26, 2020

This will be added to the next release cc: @drazisil

@thomasrockhu thomasrockhu merged commit 4ec70db into codecov:master May 26, 2020
@ofek ofek deleted the patch-1 branch May 26, 2020 22:05
@ofek
Copy link
Contributor Author

ofek commented May 26, 2020

Thank you!!!

@ofek ofek mentioned this pull request May 26, 2020
Merged
nmoinvaz added a commit to nmoinvaz/zlib-ng that referenced this pull request Jul 10, 2020
@nmoinvaz
Use mainline codecov repository since codecov/codecov-python#169 has …
a60eec8 
…been merged.
Dead2 pushed a commit to zlib-ng/zlib-ng that referenced this pull request Aug 25, 2020
@nmoinvaz @Dead2
Use mainline codecov repository since codecov/codecov-python#169 has …
1588f9d 
…been merged.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@drazisil drazisil Awaiting requested review from drazisil

1 more reviewer

@thomasrockhu thomasrockhu thomasrockhu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@drazisil drazisil

Labels

bug patched & pending review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ofek @nmoinvaz @drazisil @thomasrockhu