Update dependencies #12277
Update dependencies #12277
Conversation
| { | ||
| // Both sequences end at the same point. We are finished. | ||
| return; | ||
| } |
There was a problem hiding this comment.
nit: arguable indent of not needed { }
(not requesting a change)
There was a problem hiding this comment.
This was auto formatted by VS...
¯\_(ツ)_/¯
|
still needs ad35c81 (I could not push to this PR) |
|
With this I cannot create packages. $ powershell.exe '.\setup\installer\Check-BundlesConsistent.ps1' 'C:\dev\gc\gitextensions_4\artifacts\Release\publish\GitExtensions' |
|
Thank you, folks. I'll look into these in the coming days. |
|
MC resolved and squashed in 8bb1abe. |
🙇♂️ |
|
The build is green. |
| @@ -227,6 +224,9 @@ | |||
| <Component Id="TestableIO.System.IO.Abstractions.Wrappers.dll" Guid="*"> | |||
| <File Source="$(var.ArtifactsPublishPath)\TestableIO.System.IO.Abstractions.Wrappers.dll" /> | |||
| </Component> | |||
| <Component Id="Testably.Abstractions.FileSystem.Interface.dll" Guid="*"> | |||
There was a problem hiding this comment.
Why do Testably have to be bundled?
There was a problem hiding this comment.
It's a "new" dependency of System.IO.Abstractions.
Bump most of the dependencies. There are some dependencies which I'm not sure about, so I left those alone.
Whilst building a fresh clone I got a bunch of errors similar to the following:
This specific issue was transitively caused by the old verison of NSubstitute we use.