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.

Sign up for GitHub

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

Jump to bottom

Update MSBuild.StructuredLogger to handle newer versions #2597

Closed
wants to merge 17 commits into from
Closed

Update MSBuild.StructuredLogger to handle newer versions #2597

wants to merge 17 commits into from

Conversation

TheAngryByrd
Copy link
Contributor

@TheAngryByrd TheAngryByrd commented May 28, 2021 •
edited
Loading

Description

Closes #2595

With the latest version of dotnet sdk (5.0.300), running FAKE gets this error:

Unsupported log file format. Latest supported version is 9, the log file has version 13.

This updates MSBuild.StructuredLogger to be able to handle it. MSBuild.StructureLogger added support in 2.1.397.

matthid and others added 16 commits March 31, 2021 18:12
@matthid
Next Release
ff35d64
@yazeedobaid
add access token option to sqlpackage
b6c3a72
@yazeedobaid
Porting Fixie Module to FAKE 5
f58b526
@matthid
Merge pull request fsprojects#2581 from FoothillSolutions/add-access-…
a4b56fe 
…token-to-sqlpackage

add access token option to sqlpackage
@matthid
Merge pull request fsprojects#2582 from FoothillSolutions/port-fixie-…
8bd3485 
…module

Porting Fixie Module to FAKE 5
@yazeedobaid
Porting Bitbucket module to FAKE 5
00aec32
@matthid
Merge pull request fsprojects#2583 from FoothillSolutions/port-bitbuc…
3962129 
…ket-module

Porting Bitbucket module to FAKE 5
@yazeedobaid
Fix dotnet path on wsl
c2890e4
@yazeedobaid
Get by packageid and an api to search by title
d22b228
@matthid
Merge pull request fsprojects#2584 from FoothillSolutions/fix-dotnet-…
7d13df3 
…path-on-wsl

Fix dotnet path on wsl
@matthid
Merge pull request fsprojects#2585 from FoothillSolutions/fix-nuget-f…
9ab339b 
…eed-apis

Get by packageid and an api to search by title
@yazeedobaid
Porting TypeScript module to FAKE 5
44803ad
@yazeedobaid
fix call to ts compiler
d7eda49
@matthid
Merge pull request fsprojects#2586 from FoothillSolutions/port-typesc…
e7461b6 
…ript-module

Porting TypeScript module to FAKE 5
@lucasteles
Changing coverlet output format to receive a list of formats (fsproje…
b4e5e63 
…cts#2562)

* changing the output format to receive a list of formats

* wip

* Revert "wip"

This reverts commit 6bf28d9.
@ShalokShalom
Install Instruction (fsprojects#2578)
b805f95 
* Install Instruction

Add install instructions
@TheAngryByrd TheAngryByrd mentioned this pull request May 28, 2021
Closed
@TheAngryByrd TheAngryByrd marked this pull request as ready for review May 28, 2021 15:14
@ImaginaryDevelopment
Copy link

Is this workaround working for you? It isn't for me so far when trying to build in docker @TheAngryByrd

@TheAngryByrd
Copy link
Contributor Author

Is this workaround working for you? It isn't for me so far when trying to build in docker @TheAngryByrd

I haven’t tried but I’ll try them soon

@yazeedobaid
Copy link
Collaborator

Is this workaround working for you? It isn't for me so far when trying to build in docker @TheAngryByrd

I tried the workaround in a project and it worked.

@TheAngryByrd could you please look into the Travis build? It seems one of the integration tests is failing.
I cloned the branch and run integration tests on my machine (Windows 10) and it passes. Maybe it is something with Linux!

@yazeedobaid yazeedobaid mentioned this pull request Jul 5, 2021
Closed
nhirschey added a commit to fsprojects/FSharp.Data.Toolbox that referenced this pull request Aug 12, 2021
@nhirschey
try fix at .net 5.0.200 to fix log v9 error
622c44d 
fsprojects/FAKE#2597
@yazeedobaid
Copy link
Collaborator

Thanks for the contribution
Will close this because it is outdated and the structured logger version has been updated in the latest releases.
Please feel free to update the PR and open it again if the issue is still exists.

Thanks

@yazeedobaid yazeedobaid closed this Jan 6, 2022
@cataggar cataggar mentioned this pull request May 26, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Unsupported log file format. Latest supported version is 9, the log file has version 13.
7 participants
@TheAngryByrd @ImaginaryDevelopment @yazeedobaid @kinsb @matthid @lucasteles @ShalokShalom