Merge pull request #10 from prathameshd30/prathameshd30-git-actions

Improve CMake GitHub actions workflow
prathameshd30 · Aug 14, 2024 · f8c5a95 · f8c5a95
2 parents c133ede + fd50f69
commit f8c5a95
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/.github/workflows/cmake-multi-platform-val.yml b/.github/workflows/cmake-multi-platform-val.yml
@@ -1,6 +1,15 @@
 name: Multiplatform CMake Build, CTest, Valgrind Profiling
 
 on:
+  push:
+    branches:
+     - main
+  pull_request:
+    branches:
+     - main
+    paths-ignore:
+     - 'README.md'
+
   workflow_dispatch:
 
 jobs:
@@ -95,4 +104,4 @@ jobs:
     
 
 
-
+
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -3,8 +3,9 @@ cmake_minimum_required(VERSION 3.28) # Needed to reduce version for compatibilit
 project(prime-headers)
 
 set(CMAKE_C_STANDARD 11)
-
 set(CMAKE_C_STANDARD_REQUIRED ON)
+set(CMAKE_BUILD_TYPE Debug)
+##set(CMAKE_BUILD_TYPE Release)
 
 # deliberately not including these subdirectories by glob regex as precaution