Skip to content

dotnet10 bump! (PR 1 of 2) - #827

Open
baumannalexj wants to merge 1 commit into
OpenRA:masterfrom
baumannalexj:dotnet10
Open

baumannalexj wants to merge 1 commit into
OpenRA:masterfrom
baumannalexj:dotnet10

Conversation

@baumannalexj

@baumannalexj baumannalexj commented Sep 7, 2026 •

Copy link
Copy Markdown
  • bump to dotnet10 in preparation for updating the OpenRA version to latest
  • added a test to help when updating OpenRA (next PR)
  • .csproj updates are a side effect of new dotnet build
  • patches CVE security vulnerability

- OpenRA.Mods.RA2.csproj: make EngineRootPath a conditioned default
  (overridable via -p:EngineRootPath=... or an EngineRootPath env var)
  instead of an unconditional assignment, so the mod can be pointed at
  an arbitrary engine checkout without a ./engine symlink/junction.
- Add OpenRA.Mods.RA2.Test (NUnit), matching the engine's own
  OpenRA.Test project layout. EngineCompatibilityTests reflects over
  ChronoResourceDelivery and WithVoxelHelicopterBodyInfo to assert
  they still satisfy INotifyHarvesterAction and
  IRenderActorPreviewVoxelsInfo respectively - these traits implement
  engine-supplied interfaces directly (the trait system is the
  composition root), so there's no adapter layer to shield them from
  upstream interface changes. Verified red on release-20250330,
  green on release-20231010.
- Ignore .worktrees (used for nested engine-bump worktrees).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YSDurhbS9Xa3qhBxcHSFFb
@baumannalexj baumannalexj changed the title Add engine-compatibility tests; make EngineRootPath overridable dotnet10 bump! (PR 1 of 2) Sep 7, 2026
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.

1 participant