Remove dotnet format install command

[bdovaz]

In .NET 6 it is included in the SDK

#2310 (comment)

[nvuillam]

🦙 MegaLinter status: ⚠️ WARNING

Descriptor	Linter	Files	Fixed	Errors	Elapsed time
✅ BASH	bash-exec	7		0	0.01s
✅ BASH	shellcheck	7		0	0.39s
✅ BASH	shfmt	7	0	0	0.34s
✅ COPYPASTE	jscpd	yes		no	2.66s
✅ DOCKERFILE	hadolint	105		0	10.11s
✅ JSON	eslint-plugin-jsonc	21	0	0	2.06s
✅ JSON	jsonlint	19		0	0.25s
✅ JSON	v8r	21		0	14.22s
⚠️ MARKDOWN	markdownlint	309	0	229	6.64s
✅ MARKDOWN	markdown-link-check	309		0	5.41s
✅ MARKDOWN	markdown-table-formatter	309	0	0	18.69s
✅ OPENAPI	spectral	1		0	0.85s
⚠️ PYTHON	bandit	176		45	2.35s
✅ PYTHON	black	176	0	0	5.12s
✅ PYTHON	flake8	176		0	2.07s
✅ PYTHON	isort	176	0	0	0.74s
✅ PYTHON	mypy	176		0	7.37s
✅ PYTHON	pylint	176		0	13.41s
⚠️ PYTHON	pyright	176		276	18.91s
✅ REPOSITORY	checkov	yes		no	28.52s
✅ REPOSITORY	git_diff	yes		no	0.37s
✅ REPOSITORY	secretlint	yes		no	8.45s
✅ REPOSITORY	trivy	yes		no	23.4s
✅ SPELL	cspell	730		0	19.59s
✅ SPELL	misspell	551	0	0	0.85s
✅ XML	xmllint	3	0	0	0.33s
✅ YAML	prettier	81	0	0	2.83s
✅ YAML	v8r	23		0	62.78s
✅ YAML	yamllint	82		0	1.63s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by

[echoix]

The title may be misleading, we are removing dotnet-format (the tool), not the dotnet format.

We might as well change the "linter name", to have the documentation page not having contradictions. => would there be any obvious side effects? (Like does it break too much links, does something else depend on the exact name?)
At least the env variable name would be the same and correct, not like the poor choice of the first powershell linter that was called powershell instead of PSScriptAnalyzer.

[bdovaz]

@nvuillam about the linter_name property question do you know anything? In theory I understand that it would not generate any breaking change no? That only affects the documentation I imagine.

@echoix I guess you want to change it to linter_name: dotnet format? Or do you mean something else?

[nvuillam]

linter_name can probably be updated without major break, but I can't promise about documentation... so better not change it ^^

[nvuillam]

Good, it will decrease the size of some docker images ^^

[nvuillam]

tflint failing test case is because of github API limits , so I forced the merge ^^