-
Notifications
You must be signed in to change notification settings - Fork 325
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
Add coverlet smoke test #2348
Add coverlet smoke test #2348
Conversation
Will run
|
PR is just a pointer to a branch. You could just update the previous PR by pushing into it’s branch (even rebasing and force pushing if needed. Well next time |
I know, #2330 (comment) as explained here I tried to rebase and resolve conflicts a pair of time with no luck(repo build error due to issue on sln guids) your update was a bit invasive 😄 on project structure, so to not waste too much time I created a new one since my update are few code lines. |
In that case cherry-picking your commits over the new code might work. But I am just throwing options out there, it does not really matter 🙃
…-jakub
________________________________
Od: Marco Rossignoli <notifications@github.com>
Odesláno: Monday, March 2, 2020 9:27:16 AM
Komu: microsoft/vstest <vstest@noreply.github.com>
Kopie: Jakub Jareš <me@jakubjares.com>; Mention <mention@noreply.github.com>
Předmět: Re: [microsoft/vstest] Add coverlet smoke test (#2348)
PR is just a pointer to a branch. You could just update the previous PR by pushing into it’s branch (even rebasing and force pushing if needed. Well next time
|
Thanks advices are always well accepted! |
Replacement of #2330
cc: @nohwnd