Skip to content

Upgrade Spectre.Console to v0.46.0#4140

Merged
augustoproiete merged 2 commits intocake-build:developfrom
FrankRay78:Update-spectre.console-to-0.46.0
Apr 5, 2023
Merged

Upgrade Spectre.Console to v0.46.0#4140
augustoproiete merged 2 commits intocake-build:developfrom
FrankRay78:Update-spectre.console-to-0.46.0

Conversation

@FrankRay78
Copy link
Copy Markdown
Contributor

This PR upgrades spectre.console from 0.45.0 to 0.46.0 (latest) by upgrading the NuGet package reference, and the subsequent minor build issue introduced (see: #4138) has been fixed by a trivial code change.

Background: my interest in upgrading to the latest spectre.console is in anticipation of being able to fix this cake issue, #3279. The necessary change to spectre.console has been made and committed (714cf179cb349597a8b775287ac8299584b18617), although it does not reside in the 0.46.0 release and is scheduled for 0.47.0. Either way, I wanted to upgrade the version of spectre.console used by cake to the latest and address any issues that may arise, to ease the future upgrade to 0.47.0

nb. spectre.console release notes for 0.46.0 are here: https://github.com/spectreconsole/spectre.console/releases/tag/0.46.0

The biggest visible improvement arising from using spectre.console 0.46.0 for cake users is showing the default value of any arguments, see:

image

@FrankRay78
Copy link
Copy Markdown
Contributor Author

Looks like .LeftAligned() in AnsiConsole.Render(new FigletText("Cake").LeftAligned().Color(Color.Red)); in the integration tests is breaking the build. Perhaps a 'not-intended' breaking change in spectre.console. I'll investigate...

@patriksvensson
Copy link
Copy Markdown
Member

@FrankRay78 LeftAligned was renamed to LeftJustified.

@augustoproiete augustoproiete requested a review from a team April 5, 2023 16:11
@augustoproiete augustoproiete linked an issue Apr 5, 2023 that may be closed by this pull request
2 tasks
@augustoproiete augustoproiete changed the title Upgrade spectre.console 0.46.0 (latest) Upgrade Spectre.Console to v0.46.0 Apr 5, 2023
Copy link
Copy Markdown
Member

@augustoproiete augustoproiete left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@augustoproiete augustoproiete merged commit e2ccd15 into cake-build:develop Apr 5, 2023
@augustoproiete
Copy link
Copy Markdown
Member

@FrankRay78 your changes have been merged, thanks for your contribution 👍

@FrankRay78 FrankRay78 deleted the Update-spectre.console-to-0.46.0 branch October 18, 2023 13:21
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.

Upgrade Spectre.Console to v0.46.0

3 participants