Skip to content

ci: build managed bindings before packaging - #89

Merged
Seddryck merged 3 commits into
mainfrom
codex/issue-88-parallel-managed-build
Aug 18, 2026
Merged

ci: build managed bindings before packaging#89
Seddryck merged 3 commits into
mainfrom
codex/issue-88-parallel-managed-build

Conversation

@Seddryck

Copy link
Copy Markdown
Owner

Summary

  • add a dedicated managed-bindings job that runs independently of the native RID matrix
  • upload managed bin/Release and obj outputs for the packaging job
  • make the NuGet packaging job consume managed and native artifacts without restoring or rebuilding
  • allow CI to disable the project’s local native grammar build while preserving the default local behavior
  • rename the packaging-only script to Pack-NuGetPackage.ps1

Validation

  • dotnet restore bindings/csharp/Expressif.Syntax/Expressif.Syntax.csproj -p:Version=0.0.0-ci -p:PackageVersion=0.0.0-ci
  • dotnet build bindings/csharp/Expressif.Syntax/Expressif.Syntax.csproj --configuration Release --no-restore --disable-build-servers -m:1 -p:Version=0.0.0-ci -p:PackageVersion=0.0.0-ci -p:BuildNativeGrammar=false
  • ./scripts/Pack-NuGetPackage.ps1 -Version 0.0.0-ci -NativeAssetsDirectory native-assets with representative files for all six RIDs
  • verified the package contains managed DLL/XML outputs for net8.0, net9.0, and net10.0, plus all six native runtime assets
  • parsed .github/workflows/ci.yml with PyYAML
  • git diff --check

Closes #88

@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: f87365e3-848f-4273-b879-819117af0569


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Seddryck
Seddryck marked this pull request as ready for review August 18, 2026 21:34
@Seddryck
Seddryck merged commit d6e907e into main Aug 18, 2026
26 of 27 checks passed
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.

Build managed bindings in parallel with native libraries before packaging

1 participant