Skip to content

Fix solution build configuration#62

Open
Sean-Kenneth-Doherty wants to merge 1 commit into
zer0Kerbal:1.2.99.0-prereleasefrom
Sean-Kenneth-Doherty:fix/documentation-output-path
Open

Fix solution build configuration#62
Sean-Kenneth-Doherty wants to merge 1 commit into
zer0Kerbal:1.2.99.0-prereleasefrom
Sean-Kenneth-Doherty:fix/documentation-output-path

Conversation

@Sean-Kenneth-Doherty
Copy link
Copy Markdown

@Sean-Kenneth-Doherty Sean-Kenneth-Doherty commented May 17, 2026

Summary

  • enable FieldTrainingFacility.csproj to build for Debug and Release solution configurations
  • point XML documentation output at the repository docs/Code/Code.xml file
  • guard the text templating import and pre-build transform so the checked-in generated Version.cs remains usable when those components are not installed

Issue

Closes #61

Validation

  • git diff --check
  • xmllint --noout source/FieldTrainingFacility.csproj
  • mono /tmp/KSPMMCfgValidator.exe .
  • xbuild /nologo /t:ValidateSolutionConfiguration source/FieldTrainingFacility.sln
  • xbuild /nologo /p:Configuration=Release /p:K112=.../KSP_Data/Managed/ /p:PostBuildEvent= source/FieldTrainingFacility.sln against local KSP 1.12.5 managed assemblies; build succeeded with existing documentation/toolset warnings

No runtime/game test was performed.

@zer0Kerbal zer0Kerbal requested review from Lisias and zer0Kerbal May 18, 2026 10:50
@zer0Kerbal zer0Kerbal changed the base branch from master to 1.2.99.0-prerelease May 18, 2026 10:51
@zer0Kerbal zer0Kerbal added this to the 1.2.99.0-prerelease milestone May 18, 2026
@Sean-Kenneth-Doherty Sean-Kenneth-Doherty marked this pull request as ready for review May 18, 2026 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Solution build configuration misses project build output

3 participants