Skip to content

Commit

Permalink
Fix Windows actions. (#41)
Browse files Browse the repository at this point in the history
* Invoke CTest directly.

* Remove VS2017 build for now.

* Fix docs.
  • Loading branch information
tmadden committed Oct 8, 2020
1 parent f39d5ae commit 67c2e57
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 35 deletions.
29 changes: 2 additions & 27 deletions .github/workflows/windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,32 +13,6 @@ env:

jobs:

VS2017:
runs-on: windows-2016

strategy:
matrix:
config: [Debug, Release]

steps:
- uses: actions/checkout@v2

- name: Configure
run: |
cmake -Bbuild -DCMAKE_BUILD_TYPE=${{matrix.config}} .
- name: Build tests
run: >
cmake --build build --config ${{matrix.config}}
--target unit_test_runner -j4
cmake --build build --config ${{matrix.config}}
--target single_header_tester
- name: Test
run: |
cmake --build build --config ${{matrix.config}} --target ctest
VS2019:
runs-on: windows-2019

Expand All @@ -63,4 +37,5 @@ jobs:
- name: Test
run: |
cmake --build build --config ${{matrix.config}} --target ctest
cd build
ctest --build-config ${{matrix.config}}
13 changes: 5 additions & 8 deletions docs/getting-started.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,12 @@ The Core Library
----------------

The core of alia should work on any C++14 compiler. It has no external
dependencies. It's regularly tested on:
dependencies. The following compilers are officially supported:

- Visual C++ 2015 and 2017 (x86 and x64 for both, C++14 features only)
- GCC 5 (C++14 only) and 7 (with C++17 features)
- Clang 4.0 (C++14 only) and 5.0 (with C++17 features)
- Emscripten 1.39.7 (including C++17 features)

It should support more recent versions of the above. If you find otherwise,
please open an issue.
- Visual C++ 2015 and higher
- GCC 5 and higher
- Clang 4.0 and higher
- Emscripten 1.39.7 and higher

### Getting It

Expand Down

0 comments on commit 67c2e57

Please sign in to comment.