-
Notifications
You must be signed in to change notification settings - Fork 462
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.
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
Begin converting to GitHub Actions #381
Merged
Merged
Changes from 22 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
409c217
Begin adding Github Actions for tests
4d8ccea
working directory fix
738beb2
split out windows tests
5050bbc
consistent compiler
84b640f
Add Valgrind, etc
b45a44b
wip
c17d226
CTest output for Windows
1367d11
Run coverage on coverage
8c2888b
Run Windows tests using cmd
1475bbb
Use cmd also for configure
c41dc44
Disable Windows test running
6f30f33
Add arch to Windows build
c5ba95a
Set arch on CMake
be2c2cb
test formatting error
c85f0ec
Revert "test formatting error"
43f68e7
Remove some tests now run in Github Actions
f2623a1
Newline at end of file
b95c1cf
Fix Windows tests by switching to bash
9e6d02c
Disable Appveyor
18783bb
Merge linux based tests into one workflow
ff92851
Note use of bash in changelog
b46f45c
Try to fix OSX tests
5c60a42
Update badges
isaacbrodsky File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,112 @@ | ||
name: test-linux | ||
|
||
on: | ||
push: | ||
branches: [master] | ||
pull_request: | ||
branches: [master] | ||
|
||
jobs: | ||
tests: | ||
name: Test Compile ${{ matrix.compiler }} | ||
runs-on: ubuntu-latest | ||
env: | ||
CC: ${{ matrix.compiler }} | ||
|
||
strategy: | ||
matrix: | ||
compiler: [clang, gcc] | ||
|
||
steps: | ||
- uses: actions/[email protected] | ||
|
||
- name: Install Doxygen | ||
run: sudo apt-get install doxygen graphviz | ||
|
||
- name: Configure build | ||
run: cmake -Bbuild -DWARNINGS_AS_ERRORS=ON . | ||
|
||
- name: Formatting check | ||
working-directory: build | ||
run: | | ||
clang-format-9 --version | ||
make format | ||
git diff --exit-code | ||
|
||
- name: Build | ||
working-directory: build | ||
run: make | ||
|
||
- name: binding-functions | ||
working-directory: build | ||
run: | | ||
make binding-functions | ||
test -s binding-functions | ||
|
||
- name: Tests | ||
working-directory: build | ||
run: | | ||
make test | ||
sudo make install | ||
|
||
# Note the packages aren't used to test the examples below | ||
- name: Test packaging | ||
working-directory: build | ||
run: cpack -D CPACK_PACKAGE_CONTACT="Test build in CI" | ||
|
||
- name: Examples | ||
run: | | ||
mkdir build/examples | ||
cd build/examples | ||
cmake ../../examples | ||
make | ||
make test | ||
|
||
valgrind-tests: | ||
name: Test Valgrind | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- uses: actions/[email protected] | ||
|
||
- name: Install Valgrind | ||
run: sudo apt-get install valgrind | ||
|
||
- name: Configure build | ||
run: cmake -Bbuild -DCMAKE_BUILD_TYPE=Debug -DWRAP_VALGRIND=ON . | ||
|
||
- name: Build | ||
working-directory: build | ||
run: make | ||
|
||
- name: Tests | ||
working-directory: build | ||
env: | ||
CTEST_OUTPUT_ON_FAILURE: 1 | ||
run: make test-fast | ||
|
||
coverage-tests: | ||
name: Coverage | ||
runs-on: ubuntu-latest | ||
env: | ||
CC: gcc | ||
|
||
steps: | ||
- uses: actions/[email protected] | ||
|
||
- name: Install lcov | ||
run: sudo apt-get install lcov | ||
|
||
- name: Configure build | ||
run: cmake -DCMAKE_BUILD_TYPE=Debug -DWARNINGS_AS_ERRORS=ON -DH3_PREFIX=testprefix_ . | ||
|
||
- name: Build | ||
run: make | ||
|
||
- name: Tests | ||
run: make coverage | ||
|
||
- uses: coverallsapp/github-action@master | ||
with: | ||
path-to-lcov: ./coverage.cleaned.info | ||
github-token: ${{ secrets.GITHUB_TOKEN }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,42 @@ | ||
name: test-osx | ||
|
||
on: | ||
push: | ||
branches: [master] | ||
pull_request: | ||
branches: [master] | ||
|
||
jobs: | ||
tests: | ||
name: Test Compile ${{ matrix.compiler }} | ||
runs-on: macos-latest | ||
|
||
steps: | ||
- uses: actions/[email protected] | ||
|
||
- name: Configure build | ||
run: cmake -Bbuild -DWARNINGS_AS_ERRORS=ON . | ||
|
||
- name: Build | ||
working-directory: build | ||
run: make | ||
|
||
- name: binding-functions | ||
working-directory: build | ||
run: | | ||
make binding-functions | ||
test -s binding-functions | ||
|
||
- name: Tests | ||
working-directory: build | ||
run: | | ||
make test | ||
sudo make install | ||
|
||
- name: Examples | ||
run: | | ||
mkdir build/examples | ||
cd build/examples | ||
cmake ../../examples | ||
make | ||
make test |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
name: test-website | ||
|
||
on: | ||
push: | ||
branches: [master] | ||
pull_request: | ||
branches: [master] | ||
|
||
jobs: | ||
tests: | ||
name: Test Website and FOSSA | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- uses: actions/[email protected] | ||
|
||
- uses: actions/setup-node@v1 | ||
with: | ||
node-version: 10.x | ||
|
||
- name: Install FOSSA | ||
run: | | ||
curl -H 'Cache-Control: no-cache' https://raw.githubusercontent.com/fossas/fossa-cli/master/install.sh | sudo bash | ||
|
||
- name: Test Website Build | ||
working-directory: website | ||
run: | | ||
yarn | ||
yarn build | ||
|
||
- name: Submit FOSSA report | ||
run: if [ -n "${{ secrets.FOSSA_API_KEY }}" ]; then fossa; fi |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,44 @@ | ||
name: test-windows | ||
|
||
on: | ||
push: | ||
branches: [master] | ||
pull_request: | ||
branches: [master] | ||
|
||
jobs: | ||
tests: | ||
name: Test Compile ${{ matrix.config }} ${{ matrix.arch }} | ||
runs-on: windows-latest | ||
env: | ||
CC: cl.exe | ||
CXX: cl.exe | ||
|
||
strategy: | ||
matrix: | ||
config: [Release, Debug] | ||
arch: [Win32, x64] | ||
|
||
steps: | ||
- uses: actions/[email protected] | ||
|
||
- name: Configure build | ||
shell: cmd | ||
run: cmake -Bbuild -A ${{ matrix.arch }} -DWARNINGS_AS_ERRORS=ON . | ||
|
||
- name: Build | ||
working-directory: build | ||
run: cmake --build . --config ${{ matrix.config }} | ||
|
||
- name: binding-functions | ||
working-directory: build | ||
run: | | ||
cmake --build . --config ${{ matrix.config }} --target binding-functions | ||
if ((Get-Item "binding-functions").Length -lt 10) { $host.SetShouldExit(1) } | ||
|
||
- name: Tests | ||
working-directory: build | ||
shell: cmd | ||
env: | ||
CTEST_OUTPUT_ON_FAILURE: 1 | ||
run: ctest -C ${{ matrix.config }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we know of any Windows users that can comment on this?
I know that I have bash on my Windows VM, but is that true for those who treat it as a daily driver and use git?