Remove unnecessary null check (#17053) #571

	name: Main - CI

	on:
	# Run it on main and release pushes too, in case we merge from a branch that's not up-to-date with the target branch
	# and breaks something after merge (or if we push to main).
	push:
	paths-ignore:
	- '*/.md'
	- 'mkdocs.yml'
	- 'src/docs/*/'
	branches: [ main, release/** ]

	env:
	DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true
	DOTNET_CLI_TELEMETRY_OPTOUT: true

	jobs:
	test:
	runs-on: ${{ matrix.os }}
	name: Build & Test
	strategy:
	fail-fast: false
	matrix:
	os: [ubuntu-latest, windows-latest]
	steps:
	- uses: actions/checkout@v4
	- uses: actions/setup-node@v4
	with:
	node-version: "15"
	- uses: actions/setup-dotnet@v4
	with:
	dotnet-version: '8.0.x'
	- name: Build
	# See pr_ci.yml for the reason why we disable NuGet audit warnings.
	run: \|
	dotnet build -c Release -warnaserror /p:TreatWarningsAsErrors=true /p:RunAnalyzers=true /p:NuGetAudit=false
	- name: Unit Tests
	run: \|
	dotnet test -c Release --no-build ./test/OrchardCore.Tests/OrchardCore.Tests.csproj
	- name: Functional Tests
	if: matrix.os == 'ubuntu-latest'
	run: \|
	cd test/OrchardCore.Tests.Functional
	npm install
	npm run cms:test
	npm run mvc:test
	- uses: actions/upload-artifact@v4
	if: matrix.os == 'ubuntu-latest' && failure()
	with:
	name: Functional Test failure
	path: \|
	test/OrchardCore.Tests.Functional/cms-tests/cypress/screenshots
	src/OrchardCore.Cms.Web/App_Data_Tests/logs

Provide feedback