Fixed logic in ItemListBase.FindByValue to not rely on try-catch. #127

	name: build-test

	on:
	push:
	branches:
	- master
	pull_request:
	workflow_dispatch:

	jobs:
	build:
	timeout-minutes: 60
	runs-on: windows-latest

	steps:
	- name: Checkout code
	uses: actions/checkout@v4
	with:
	fetch-depth: '0'

	- name: Build project
	run: dotnet build -bl:build.binlog --configuration Release

	# there are cases where this will fail and we want to know about it
	# so we don't use continue-on-error, but we still want to publish the results
	- name: Test project
	id: test
	run: dotnet test -bl:test.binlog --filter "TestCategory != SkipOnTeamCity" --blame-hang-timeout 5m --logger:"trx;LogFilePrefix=results" --results-directory ./test-results

	- name: Publish test results
	if: ${{ !cancelled() && steps.test.outcome != 'skipped' }}
	uses: EnricoMi/publish-unit-test-result-action/windows@v2
	with:
	check_name: LibPalaso Tests
	files: ./test-results/*/.trx
	action_fail: true
	action_fail_on_inconclusive: true

	- name: Publish logs on failure
	if: failure()
	uses: actions/upload-artifact@v3
	with:
	name: binary-logs
	path: \|
	build.binlog
	test.binlog

Provide feedback