Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bazel-build for C# language pack #16519

Merged
merged 14 commits into from
May 27, 2024
Merged

Bazel-build for C# language pack #16519

merged 14 commits into from
May 27, 2024

Conversation

criemen
Copy link
Collaborator

@criemen criemen commented May 17, 2024

This commits a bazel-based build system for C# using rules_dotnet. External dependencies are managed via paket, and updates to the generated bazel files are done via ./update-deps.sh. We're providing our own (minimal) test runner for xunit
tests.

I'll follow up with an internal PR that updates the documentation on how to update dependencies.

@criemen criemen added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label May 17, 2024
@criemen criemen requested a review from redsun82 May 17, 2024 11:01
@github-actions github-actions bot added the C# label May 17, 2024
This commits a bazel-based build system for C#
using `rules_dotnet`. External dependencies are managed
via `paket`, and updates to the generated bazel files
are done via `./update-deps.sh`.
We're providing our own (minimal) test runner for `xunit`
tests.
GitInfo doesn't work with the bazel-based build.
Instead, we pull in the information from bazel,
which correctly works with the bazel cache.
@criemen criemen marked this pull request as ready for review May 21, 2024 07:15
@criemen criemen requested review from a team as code owners May 21, 2024 07:15
@criemen criemen changed the title WIP: Bazel-build for C# language pack Bazel-build for C# language pack May 21, 2024
Copy link
Contributor

@redsun82 redsun82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't tell much about C# changes, here's a first pass on my side.

.bazelrc Show resolved Hide resolved
misc/bazel/csharp.bzl Outdated Show resolved Hide resolved
misc/bazel/csharp.bzl Outdated Show resolved Hide resolved
misc/bazel/csharp.bzl Outdated Show resolved Hide resolved
tamasvajk
tamasvajk previously approved these changes May 24, 2024
Copy link
Contributor

@redsun82 redsun82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this, looks good to me on the bazel side. I would like to cut the dependencies of codeql to the internal workspace, but that's out of scope in this PR.

@criemen criemen merged commit bd6e233 into main May 27, 2024
23 of 26 checks passed
@criemen criemen deleted the criemen/bazel-csharp branch May 27, 2024 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C# depends on internal PR This PR should only be merged in sync with an internal Semmle PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants