Refresh website

[asbjornu]

Description

This PR refreshes the website to match the logo created by @LazaroOnline in #2508. It also updates the documentation so it resolves #2326.

Related Issue

• Builds on [Improvement] Make the Nuget package icon in a solid bright color with borders so it can be seen in Visual Studio's dark UI. #2508.
• Resolves [Improvement] Detailing package alternatives #2326.
• Resolves [Improvement] Document Git repository requirements #2194.
• Resolves [Docs] Update the docs to reflect the current state of the code base #2503.
• Resolves [Improvement] Document that ignore only affects the version source #2334.
• Resolves GitVersion should rely on existing dotnet SDKs rather than downloading it using brew #2551.
• Resolves [Docs] How to install on ubuntu. Why so little documentation on Linux installation? #2365.

Motivation and Context

The documentation was chaotic and outdated. This PR does not complete that work, but it improves it a whole lot.

How Has This Been Tested?

• I've made the arguments documentation shared between command line and the docs.
• I've added HTMLProofer tests that verify links in the generated documentation.

Screenshots (if appropriate):

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[asbjornu]

I'm having some Cake problems:

Cake Setup
   at Cake.Core.Tooling.Tool`1.Run(TSettings settings, ProcessArgumentBuilder arguments, ProcessSettings processSettings, Action`1 postAction) in C:\projects\cake\src\Cake.Core\Tooling\Tool.cs:line 103
   at Cake.Common.Tools.GitVersion.GitVersionRunner.Run(GitVersionSettings settings) in C:\projects\cake\src\Cake.Common\Tools\GitVersion\GitVersionRunner.cs:line 59
   at Cake.Common.Tools.GitVersion.GitVersionAliases.GitVersion(ICakeContext context, GitVersionSettings settings) in C:\projects\cake\src\Cake.Common\Tools\GitVersion\GitVersionAliases.cs:line 121
   at Submission#0.GitVersion(GitVersionSettings settings)
   at Submission#0.GetVersion(BuildParameters parameters)
   at Submission#0.<>c__DisplayClass0_6.<<Initialize>>b__114()
	Message:	GitVersion: Process returned an error (exit code 1).
	Data:	[  ]
	InnerException:	
	HelpLink:	
	Source:	Cake.Core
	HResult:	-2146233088

And:

Teardown error: System.NullReferenceException: Object reference not set to an instance of an object.
   at Cake.Core.CakeDataService.Get[TData]() in C:\projects\cake\src\Cake.Core\CakeDataService.cs:line 36
   at Cake.Core.CakeEngineActions.<>c__DisplayClass21_0`1.<RegisterTeardown>b__0(ITeardownContext context) in C:\projects\cake\src\Cake.Core\CakeEngineActions.cs:line 64
   at Cake.Core.DefaultExecutionStrategy.PerformTeardown(Action`1 action, ITeardownContext teardownContext) in C:\projects\cake\src\Cake.Core\DefaultExecutionStrategy.cs:line 57
   at Cake.Core.CakeEngine.PerformTeardown(IExecutionStrategy strategy, ICakeContext context, Stopwatch stopWatch, CakeReport report, Boolean exceptionWasThrown, Exception thrownException) in C:\projects\cake\src\Cake.Core\CakeEngine.cs:line 436

Ideas, @gep13, @arturcic?

[gep13]

@asbjornu what command are you running when you get this error?

[arturcic]

@gep13 ./build.ps1 -Target Build-Documentation

[gep13]

Ah gotcha, you mean this error is coming up when running the new docs workflow? On phone, so hard to view that.

Could it be due to not everything being checked up? In the build.hmm there is an additional checkout step, which is likely required to fully assert version number.

Arguably this isn't needed for a docs build, so could be that we make that part of the Setup task conditional?

[asbjornu]

Arguably this isn't needed for a docs build, so could be that we make that part of the Setup task conditional?

Good point, @gep13. If I only knew where Task("Setup") was defined… 😅

[gep13]

@asbjornu this isn't a Task, but rather a setup method:

https://github.com/gittools/gitversion/blob/main/build.cake#L55

[asbjornu]

Gotcha @gep13. Thanks!

[arturcic]

@asbjornu look really great the new UI, really fresh. I had a quick review, there is one comment I added but overall looks good. I think we can merge it as it is now, later we can/need to improve with follow up PRs