Skip to content

Commit

Permalink
Merge pull request #34 from coders-school/separate_workflows
Browse files Browse the repository at this point in the history
separate pre-commit workflow
  • Loading branch information
Vicx95 authored Mar 6, 2024
2 parents 9b8b56b + 0826bdc commit 7bd1079
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 10 deletions.
11 changes: 1 addition & 10 deletions .github/workflows/ci_basic.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,15 +35,6 @@ jobs:
# Build your program with the given configuration
run: cmake --build ${{env.BUILD_PATH}} --config ${{env.BUILD_TYPE}}

- name: Install dependencies
run: |
sudo apt-get update
sudo apt-get install -y llvm cppcheck
sudo apt-get install -y iwyu
- name: Pre-commit actions
uses: pre-commit/[email protected]

- name: Configure Test CMake
run: cmake -B ${{env.TEST_BUILD_PATH}} -S ${{env.TEST_SOURCE}} -DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}}

Expand All @@ -54,4 +45,4 @@ jobs:
- name: Test
working-directory: ${{env.TEST_BUILD_PATH}}
run: ctest -C ${{env.BUILD_TYPE}}


23 changes: 23 additions & 0 deletions .github/workflows/pre-commit.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
name: Pre-commit ubuntu

on: [push, pull_request]

env:
BUILD_TYPE: Release

jobs:
pre-commit:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v3

- name: Install dependencies
run: |
sudo apt-get update
sudo apt-get install -y llvm cppcheck
sudo apt-get install -y iwyu
- name: Pre-commit actions
uses: pre-commit/[email protected]

0 comments on commit 7bd1079

Please sign in to comment.