Skip to content

Commit ef2ac50

Browse files
committed
2 parents 2e243b7 + 82af509 commit ef2ac50

5 files changed

Lines changed: 36 additions & 14 deletions

File tree

.github/workflows/cygwin.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ jobs:
2020
- uses: actions/checkout@v4
2121

2222
- name: Set up Cygwin
23-
uses: cygwin/cygwin-install-action@v4
23+
uses: cygwin/cygwin-install-action@v6
2424
with:
2525
packages: cmake libboost-devel libtool gcc-g++ binutils make
2626

.github/workflows/github_actions_build.yml

Lines changed: 24 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ jobs:
1111
runs-on: ubuntu-latest
1212
steps:
1313
- uses: actions/checkout@master
14-
- uses: DoozyX/clang-format-lint-action@v0.17
14+
- uses: DoozyX/clang-format-lint-action@v0.20
1515
with:
1616
source: '.'
1717
exclude: './third_party'
@@ -130,7 +130,7 @@ jobs:
130130
CXX: ${{ matrix.cxx-compiler }}
131131
steps:
132132
- uses: actions/checkout@master
133-
- uses: seanmiddleditch/gha-setup-ninja@v4
133+
- uses: seanmiddleditch/gha-setup-ninja@v6
134134
if: contains(matrix.generator, 'Ninja')
135135

136136
- name: install boost
@@ -175,7 +175,7 @@ jobs:
175175
DEVELOPER_DIR: '/Applications/Xcode_${{ matrix.xcode-version }}.app/Contents/Developer'
176176
steps:
177177
- uses: actions/checkout@master
178-
- uses: seanmiddleditch/gha-setup-ninja@v4
178+
- uses: seanmiddleditch/gha-setup-ninja@v6
179179
if: contains(matrix.generator, 'Ninja')
180180

181181
- name: install boost
@@ -192,13 +192,13 @@ jobs:
192192
fail-fast: false
193193
matrix:
194194
include:
195-
- job-name: WIN-2025-VS-2019
195+
- job-name: WIN-2025-VS-2022
196196
os-version: '2025'
197-
generator: '"Visual Studio 16 2019"'
197+
generator: '"Visual Studio 17 2022"'
198198

199-
- job-name: WIN-2025-VS-2019 (clangcl)
199+
- job-name: WIN-2025-VS-2022 (clangcl)
200200
os-version: '2025'
201-
generator: '"Visual Studio 16 2019" -T "clangcl"'
201+
generator: '"Visual Studio 17 2022" -T "clangcl"'
202202

203203
- job-name: WIN-latest-VS-2022
204204
os-version: 'latest'
@@ -244,3 +244,20 @@ jobs:
244244
run: |
245245
cd ${{ env.BUILD_PATH }}
246246
ctest --verbose --output-on-failure -C Debug
247+
248+
auto-merge:
249+
needs: [Linux, macOS, Windows]
250+
runs-on: ubuntu-latest
251+
steps:
252+
- name: Check out repo
253+
uses: actions/checkout@v4
254+
- name: auto-merge
255+
if: |
256+
github.actor == 'dependabot[bot]' &&
257+
github.event_name == 'pull_request'
258+
run: |
259+
gh pr merge --auto --rebase "$PR_URL"
260+
env:
261+
PR_URL: ${{github.event.pull_request.html_url}}
262+
# this secret needs to be in the settings.secrets.dependabot
263+
GITHUB_TOKEN: ${{secrets.GH_ACTION_TOKEN}}

ApprovalTests/integrations/catch/Catch2Approvals.h

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,10 @@ struct Catch2ApprovalListener : Catch::TestEventListenerBase
4848

4949
virtual void sectionEnded(Catch::SectionStats const& /*sectionStats*/) override
5050
{
51-
currentTest.sections.pop_back();
51+
if (!currentTest.sections.empty())
52+
{
53+
currentTest.sections.pop_back();
54+
}
5255
}
5356
};
5457
//}

ApprovalTests/integrations/fmt/FmtToString.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,15 @@
11
#pragma once
22
#ifdef FMT_VERSION
3+
4+
#include <fmt/format.h> // for fmt::to_string()
5+
36
namespace ApprovalTests
47
{
58
class FmtToString
69
{
710
public:
811
template <typename T> static std::string toString(const T& printable)
912
{
10-
(void)printable;
1113
return fmt::to_string(printable);
1214
}
1315
};

doc/CMakeIntegration.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ It also needs CMake 3.14 or above.
264264
We use this in our `CMakeLists.txt` file:
265265

266266
<!-- snippet: cpm_download -->
267-
<a id='snippet-cpm_download'></a>
267+
<a id='snippet-https://raw.githubusercontent.com/Laguna1989/OpenALpp/master/ext/CMakeLists.txt%23cpm_download'></a>
268268
```txt
269269
message(STATUS "Add CPM.cmake")
270270
# download CPM.cmake
@@ -276,11 +276,11 @@ file(
276276
)
277277
include(${CMAKE_BINARY_DIR}/cmake/CPM.cmake)
278278
```
279-
<sup><a href='#snippet-cpm_download' title='Start of snippet'>anchor</a></sup>
279+
<sup><a href='https://raw.githubusercontent.com/Laguna1989/OpenALpp/master/ext/CMakeLists.txt#cpm_download' title='Snippet source file'>anchor</a></sup>
280280
<!-- endSnippet -->
281281

282282
<!-- snippet: cpm_add_approvaltests -->
283-
<a id='snippet-cpm_add_approvaltests'></a>
283+
<a id='snippet-https://raw.githubusercontent.com/Laguna1989/OpenALpp/master/ext/CMakeLists.txt%23cpm_add_approvaltests'></a>
284284
```txt
285285
message(STATUS "Fetching approvaltests")
286286
CPMAddPackage(
@@ -289,7 +289,7 @@ CPMAddPackage(
289289
GIT_TAG master
290290
)
291291
```
292-
<sup><a href='#snippet-cpm_add_approvaltests' title='Start of snippet'>anchor</a></sup>
292+
<sup><a href='https://raw.githubusercontent.com/Laguna1989/OpenALpp/master/ext/CMakeLists.txt#cpm_add_approvaltests' title='Snippet source file'>anchor</a></sup>
293293
<!-- endSnippet -->
294294

295295
Note the `GIT_TAG` values: This tells CMake which revision of dependencies to use. The value can be a tag or a git commit ID. Here we use `master`. However, it is generally recommended to pin your dependencies to specific versions, and test behaviour before updating to newer versions.

0 commit comments

Comments
 (0)