Skip to content

feat(ci): Use conan build to unify install, build and test this project #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
FeignClaims wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat(ci): Use conan build to unify install, build and test this project #12

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3f0d4b5
refactor(ci): Unify install, build and test into one
FeignClaims Jun 8, 2024
6e1c277
fix(ci): Unify install, build and test in codeql
FeignClaims Jun 8, 2024
99af480
fix(ci): Specify bash when running `conan build`
FeignClaims Jun 8, 2024
fdc1ea3
fix(ci): Disable other static analysis tools in codeql
FeignClaims Jun 8, 2024
67d218c
fix(ci): Build missing conan packages
FeignClaims Jun 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions .github/actions/setup_conan/action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,11 +89,6 @@ runs:
run: |
conan remove '*/system' -c

- name: Install conan packages
shell: bash
run: |
conan install . ${{ steps.output.outputs.args }} -b missing

- name: Clean 30 days unused conan caches
shell: bash
run: |
Expand Down
11 changes: 5 additions & 6 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,13 +133,12 @@ jobs:
compiler_version: ${{ steps.parsed.outputs.compiler_version }}
build_type: ${{ matrix.build_type }}

- name: Configure cmake
- name: Configure, build and test
shell: bash
run: |
cmake --preset ${{ steps.parsed.outputs.compiler_name }} ${{ steps.parsed.outputs.compiler_name == 'msvc' && '-A x64 -T v143' || '' }} -D ENABLE_DEVELOPER_MODE:BOOL=${{ matrix.developer_mode }} -D OPT_ENABLE_COVERAGE:BOOL=${{ matrix.build_type == 'Debug' }}

- name: Build and test
run: |
conan build . ${{ steps.conan.outputs.args }}
conan build . ${{ steps.conan.outputs.args }} -b missing \
-c "tools.cmake.cmaketoolchain:extra_variables*={'ENABLE_DEVELOPER_MODE': {'value': '${{ matrix.developer_mode }}', 'cache': True, 'type': 'BOOL', 'docstring': 'Set up defaults for a developer of the project, and let developer change options'}}" \
-c "tools.cmake.cmaketoolchain:extra_variables*={'OPT_ENABLE_COVERAGE' : {'value': '${{ matrix.build_type == 'Debug' }}', 'cache': True, 'type': 'BOOL', 'docstring': 'Analyze and report on coverage'}}"

- name: Unix - Coverage
if: runner.os != 'Windows' && matrix.build_type == 'Debug'
Expand Down
14 changes: 9 additions & 5 deletions .github/workflows/codeql.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,6 @@ jobs:
uses: ./.github/actions/setup_cache
with:
key: ${{ steps.parsed.outputs.os_name }}-${{ steps.parsed.outputs.os_version }}-${{ steps.parsed.outputs.compiler_name }}-${{ steps.parsed.outputs.compiler_version }}-${{ matrix.build_type }}-${{ matrix.developer_mode }}
restore_key: ${{ steps.parsed.outputs.os_name }}-${{ steps.parsed.outputs.os_version }}-${{ steps.parsed.outputs.compiler_name }}-${{ steps.parsed.outputs.compiler_version }}-${{ matrix.build_type }}

- name: Setup Cpp
uses: aminya/setup-cpp@v0.37.0
Expand Down Expand Up @@ -102,15 +101,20 @@ jobs:
# Learnt from https://github.com/igraph/igraph/issues/1181
args: -c tools.build:cflags+=-static-libasan -c tools.build:cxxflags+=-static-libasan

- name: Configure cmake
run: |
cmake --preset ${{ steps.parsed.outputs.compiler_name }} ${{ steps.parsed.outputs.compiler_name == 'msvc' && '-A x64 -T v143' || '' }} -D ENABLE_DEVELOPER_MODE:BOOL=${{ matrix.developer_mode }} -D OPT_ENABLE_COVERAGE:BOOL=${{ matrix.build_type == 'Debug' }} -D OPT_ENABLE_CLANG_TIDY:BOOL=OFF -D OPT_ENABLE_CPPCHECK:BOOL=OFF -D OPT_ENABLE_INCLUDE_WHAT_YOU_USE:BOOL=OFF

- name: Initialize CodeQL
uses: github/codeql-action/init@v3
with:
languages: cpp

- name: Configure, build and test
shell: bash
run: |
conan build . ${{ steps.conan.outputs.args }} -b missing \
-c "tools.cmake.cmaketoolchain:extra_variables*={'ENABLE_DEVELOPER_MODE' : {'value': '${{ matrix.developer_mode }}', 'cache': True, 'type': 'BOOL', 'docstring': 'Set up defaults for a developer of the project, and let developer change options'}}" \
-c "tools.cmake.cmaketoolchain:extra_variables*={'OPT_ENABLE_CLANG_TIDY' : {'value': 'OFF', 'cache': True, 'type': 'BOOL', 'docstring': 'Enable clang-tidy analysis during compilation'}}" \
-c "tools.cmake.cmaketoolchain:extra_variables*={'OPT_ENABLE_CPPCHECK' : {'value': 'OFF', 'cache': True, 'type': 'BOOL', 'docstring': 'Enable cppcheck analysis during compilation'}}" \
-c "tools.cmake.cmaketoolchain:extra_variables*={'OPT_ENABLE_INCLUDE_WHAT_YOU_USE': {'value': 'OFF', 'cache': True, 'type': 'BOOL', 'docstring': 'Enable include-what-you-use analysis during compilation'}}"

- name: Build and test
run: |
conan build . ${{ steps.conan.outputs.args }}
Expand Down