Merge pull request #5 from stanford-ssi/setup/linting-action

Setup/linting action
stanford-ssi · Oct 26, 2024 · bb06f16 · bb06f16
2 parents 58d8f7c + a590399
commit bb06f16
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/.github/workflows/c_lint.yaml b/.github/workflows/c_lint.yaml
@@ -18,7 +18,7 @@ jobs:
           format-review: true
           passive-reviews: true
       - name: Fail if there are linter errors
-        if: ${{steps.linter.outputs.clang-format-checks-failed}}
+        if: ${{steps.linter.outputs.clang-format-checks-failed > 0}}
         run: |
            echo "Linter errors occurred, failing the job."
            exit 1 # This will cause the step to fail
diff --git a/src/main.c b/src/main.c
@@ -24,8 +24,7 @@ slate_t slate;
 int main()
 {
     // Some ugly code with linter errors
-    int x = 10
-    + 5;
+    int x = 10 + 5;
     stdio_init_all();
 
     /*

diff --git a/src/scheduler/scheduler.c b/src/scheduler/scheduler.c
@@ -115,7 +115,7 @@ void sched_dispatch(slate_t *slate)
     /*
      * Transition to the next state, if required.
      */
-    sched_state_t * const next_state = current_state_info->get_next_state(slate);
+    sched_state_t *const next_state = current_state_info->get_next_state(slate);
     if (next_state != current_state_info)
     {
         LOG_INFO("sched: Transitioning to state %s", next_state->name);

diff --git a/src/state_machine/states/states.h b/src/state_machine/states/states.h
@@ -24,7 +24,7 @@ extern sched_state_t running_state;
  * nearer the top have higher priority.
  */
 static const sched_state_t *all_states[] = {&init_state, &running_state};
-static sched_state_t * const initial_state = &init_state;
+static sched_state_t *const initial_state = &init_state;
 
 /*
  * Must be a macro because it is used to initialize an array