Report DacModelExceptions better, and continue processing all files

[ErikEJ]

fixes #595

[ErikEJ]

[ErikEJ]

Build started at 07:30...
1>------ Build started: Project: TestProjectWithErrors, Configuration: Debug Any CPU ------
1>Using target framework net8.0 to run DacpacTool
1>Using package name TestProjectWithErrors and version 1.0.0
1>Using SQL Server version Sql150
1>Adding C:\Code\Github\MSBuild.Sdk.SqlProj\test\TestProjectWithErrors\Tables\MyTable.sql to the model
1>MyTable.sql(1,1): error SQL71006: Only one statement is allowed per batch. A batch separator, such as 'GO', might be required between statements.
1>Adding C:\Code\Github\MSBuild.Sdk.SqlProj\test\TestProjectWithErrors\Tables\MyTable2.sql to the model
1>MyTable2.sql(1,1): error SQL71006: Only one statement is allowed per batch. A batch separator, such as 'GO', might be required between statements.
1>C:\Code\Github\MSBuild.Sdk.SqlProj\src\MSBuild.Sdk.SqlProj\Sdk\Sdk.targets(243,5): error MSB3073: The command "dotnet "C:\Code\Github\MSBuild.Sdk.SqlProj\src\MSBuild.Sdk.SqlProj\Sdk\../tools/net8.0/DacpacTool.dll" build -o "obj\Debug\netstandard2.0\TestProjectWithErrors.dacpac" -n "TestProjectWithErrors" -v "1.0.0" -sv Sql150 -i "obj\Debug\netstandard2.0\TestProjectWithErrors.InputFiles.txt"    -dp IncludeCompositeObjects=true           " exited with code 1.
1>Done building project "TestProjectWithErrors.csproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
========== Build completed at 07:30 and took 01,579 seconds ==========

[ErikEJ]

Phew - that was an "interesting" breaking SQL container change 😄

[ErikEJ]

@jmezach All lights are green now 😄

[jmezach]

Looks like I can't approve this from my phone. Will do so when I get back home in about half an hour ;)

[jmezach]

Phew - that was an "interesting" breaking SQL container change 😄

Looks like we weren't the only ones hit by this breaking change: microsoft/mssql-docker#892

[jmezach]

Let's get this merged, nice fix I think :)

[ErikEJ]

@jmezach Agree, this is a good quality improvement

[jmezach]

Should we release this as 2.8.1?

[ErikEJ]

I have a plan for a PR to run some .NET analyzers against the DacpacTool codebase (our code, not the modules) - maybe after that?

[jmezach]

Sure thing :)