Skip to content

Commit c5b2b0d

Browse files
authored
build: Run format and clippy during C# binding build (#451)
1 parent fc09802 commit c5b2b0d

2 files changed

Lines changed: 26 additions & 4 deletions

File tree

.github/workflows/test-csharp.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,18 @@ jobs:
4141
uses: actions/checkout@v4
4242
with:
4343
fetch-depth: 0
44+
45+
- name: Fetch crates
46+
run: cargo fetch
47+
working-directory: ./bindings/ffi
48+
49+
- name: Check Regorus binding formatting
50+
run: cargo fmt --check
51+
working-directory: ./bindings/ffi
52+
53+
- name: Check Clippy linting for Regorus binding
54+
run: cargo clippy --frozen -- -D warnings
55+
working-directory: ./bindings/ffi
4456

4557
- name: Build Regorus binding
4658
run: cargo build -r --target ${{ matrix.runtime.target }} --locked

bindings/ffi/src/lib.rs

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -462,8 +462,13 @@ pub extern "C" fn regorus_engine_get_ast_as_json(engine: *mut RegorusEngine) ->
462462
/// See https://docs.rs/regorus/latest/regorus/coverage/struct.Engine.html#method.get_policy_package_names
463463
#[no_mangle]
464464
#[cfg(feature = "azure_policy")]
465-
pub extern "C" fn regorus_engine_get_policy_package_names(engine: *mut RegorusEngine) -> RegorusResult {
466-
let output = || -> Result<String> { serde_json::to_string_pretty(&to_ref(engine)?.engine.get_policy_package_names()?).map_err(anyhow::Error::msg) }();
465+
pub extern "C" fn regorus_engine_get_policy_package_names(
466+
engine: *mut RegorusEngine,
467+
) -> RegorusResult {
468+
let output = || -> Result<String> {
469+
serde_json::to_string_pretty(&to_ref(engine)?.engine.get_policy_package_names()?)
470+
.map_err(anyhow::Error::msg)
471+
}();
467472
match output {
468473
Ok(out) => RegorusResult {
469474
status: RegorusStatus::RegorusStatusOk,
@@ -479,8 +484,13 @@ pub extern "C" fn regorus_engine_get_policy_package_names(engine: *mut RegorusEn
479484
/// See https://docs.rs/regorus/latest/regorus/coverage/struct.Engine.html#method.get_policy_parameters
480485
#[no_mangle]
481486
#[cfg(feature = "azure_policy")]
482-
pub extern "C" fn regorus_engine_get_policy_parameters(engine: *mut RegorusEngine) -> RegorusResult {
483-
let output = || -> Result<String> { serde_json::to_string_pretty(&to_ref(engine)?.engine.get_policy_parameters()?).map_err(anyhow::Error::msg) }();
487+
pub extern "C" fn regorus_engine_get_policy_parameters(
488+
engine: *mut RegorusEngine,
489+
) -> RegorusResult {
490+
let output = || -> Result<String> {
491+
serde_json::to_string_pretty(&to_ref(engine)?.engine.get_policy_parameters()?)
492+
.map_err(anyhow::Error::msg)
493+
}();
484494
match output {
485495
Ok(out) => RegorusResult {
486496
status: RegorusStatus::RegorusStatusOk,

0 commit comments

Comments
 (0)