Brightspace / rules_csharp

Bazel rules for C#
Apache License 2.0
8 stars 5 forks source link

Add some basic CI #36

Closed j3parker closed 5 years ago

j3parker commented 5 years ago

Fixes #18

j3parker commented 5 years ago

I need to mount the source code from the checkout job into the Bazel container. I haven't used container: before so I need to read the docs to refresh my memory on how the paths work in actions.

j3parker commented 5 years ago

Couldn't find a valid ICU package installed on the system. Set the configuration flag System.Globalization.Invariant to true if you want to run with no globalization support.

Neat!

j3parker commented 5 years ago

Ah, setting that environment variable in the container doesn't work because the action runs in a "sandbox" without those envvars, right.

j3parker commented 5 years ago

https://bazel.build/designs/2016/06/21/environment.html

j3parker commented 5 years ago

Something is better than nothing. I can set this as a required check. I can work on bazel build in a draft PR separately.

j3parker commented 5 years ago

😮 get those characters away from CI, you'll break it!!!!