Skip to content

Commit

Permalink
Update build/test Workflows
Browse files Browse the repository at this point in the history
- Add libyaml-cpp-dev, libreadline
- Add a step to install cogapp for cog
- Use specific compiler to avoid nuisance 0-struct eerrors
- Remove clang if not using it
- Add setting up GTest in workflow
- Had to add PyYaml for unit tests
  • Loading branch information
dwai-wai committed Jan 6, 2025
1 parent 401317b commit e13f5d3
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 10 deletions.
15 changes: 8 additions & 7 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,17 +12,18 @@ jobs:
runs-on: ubuntu-24.04
steps:
- uses: actions/checkout@v4
- uses: egor-tensin/setup-clang@v1
with:
version: latest
platform: x64
- uses: ConorMacBride/install-package@v1
with:
apt: libreadline-dev
apt: gcc-11 g++-11 libreadline-dev libyaml-cpp-dev
version: latest
platform: x64
- run: |
pip install cogapp
- run: |
mkdir -p build
cd build
cmake ..
make
cmake -DBUILD_TESTS=OFF ..
make
env:
CC: gcc-11
CXX: g++-11
13 changes: 10 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,16 +12,23 @@ jobs:
runs-on: ubuntu-24.04
steps:
- uses: actions/checkout@v4
- uses: egor-tensin/setup-clang@v1
- uses: Bacondish2023/setup-googletest@v1
with:
tag: release-1.11.0
- uses: ConorMacBride/install-package@v1
with:
apt: valgrind libreadline-dev
apt: gcc-11 g++-11 libreadline-dev libyaml-cpp-dev valgrind
version: latest
platform: x64
- run: |
pip install cogapp pyyaml
- run: |
mkdir -p build
cd build
cmake -DBUILD_JSD_TESTS=ON -DCMAKE_BUILD_TYPE=Debug ..
make
make test
env CTEST_OUTPUT_ON_FAILURE=1 make -d memcheck
env CTEST_OUTPUT_ON_FAILURE=1 make -d memcheck
env:
CC: gcc-11
CXX: g++-11

0 comments on commit e13f5d3

Please sign in to comment.