Skip to content
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

Fix break in tests from VS 17.3 #804

Merged
merged 1 commit into from Sep 5, 2022
Merged

Fix break in tests from VS 17.3 #804

merged 1 commit into from Sep 5, 2022

Conversation

AArnott
Copy link
Collaborator

@AArnott AArnott commented Sep 5, 2022

Fix break in tests from VS 17.3

This is a workaround suggested in dotnet/msbuild#7873

@AArnott AArnott changed the title fixbuildbreak Fix break in tests from VS 17.3 Sep 5, 2022
@codecov
Copy link

codecov bot commented Sep 5, 2022

Codecov Report

❗ No coverage uploaded for pull request base (v3.5@6e793d6). Click here to learn what that means.
Patch has no changes to coverable lines.

❗ Current head 4c626dd differs from pull request most recent head fa71947. Consider uploading reports for the commit fa71947 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             v3.5     #804   +/-   ##
=======================================
  Coverage        ?   77.31%           
=======================================
  Files           ?       73           
  Lines           ?     3830           
  Branches        ?        0           
=======================================
  Hits            ?     2961           
  Misses          ?      869           
  Partials        ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

This is a workaround suggested in dotnet/msbuild#7873
@AArnott AArnott changed the base branch from main to v3.5 September 5, 2022 08:21
@AArnott AArnott added this to the v3.5 milestone Sep 5, 2022
@AArnott AArnott merged commit 4b38f20 into v3.5 Sep 5, 2022
@AArnott AArnott deleted the fixbuildbreak branch September 5, 2022 08:42
brighid-bot added a commit to cythral/brighid-commands that referenced this pull request Sep 20, 2022
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[Nerdbank.GitVersioning](https://togithub.com/dotnet/Nerdbank.GitVersioning)
| nuget | patch | `3.5.109` -> `3.5.113` |

---

### Release Notes

<details>
<summary>dotnet/Nerdbank.GitVersioning</summary>

###
[`v3.5.113`](https://togithub.com/dotnet/Nerdbank.GitVersioning/releases/v3.5.113)

#### What's Changed

- DeltaStreamReader should interpret instruction with size 0 as 0x10000
by [@&#8203;marcin-krystianc](https://togithub.com/marcin-krystianc) in
dotnet/Nerdbank.GitVersioning#818

#### New Contributors

- [@&#8203;marcin-krystianc](https://togithub.com/marcin-krystianc) made
their first contribution in
dotnet/Nerdbank.GitVersioning#818

**Full Changelog**:
dotnet/Nerdbank.GitVersioning@v3.5.112...v3.5.113

###
[`v3.5.112`](https://togithub.com/dotnet/Nerdbank.GitVersioning/releases/v3.5.112)

#### What's Changed

- Fix break in tests from VS 17.3 by
[@&#8203;AArnott](https://togithub.com/AArnott) in
dotnet/Nerdbank.GitVersioning#804
- Fix `Stream.Seek` implementations to reliably shift position as
required by [@&#8203;AArnott](https://togithub.com/AArnott) in
dotnet/Nerdbank.GitVersioning#816

**Full Changelog**:
dotnet/Nerdbank.GitVersioning@v3.5.109...v3.5.112

</details>

---

### Configuration

📅 **Schedule**: At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click
this checkbox.

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant