Skip to content

Further GHA CI corrections - #68

Open
chcg wants to merge 2 commits into
Governikus:communityfrom
chcg:gh_junit_test
Open

Further GHA CI corrections#68
chcg wants to merge 2 commits into
Governikus:communityfrom
chcg:gh_junit_test

Conversation

@chcg

@chcg chcg commented Jun 18, 2026

Copy link
Copy Markdown
Contributor
  • usage of vcpkg for windows llhttp build (remaining issue with yamllint), apt package for linux and brew for macos
  • use env vars also for DCMAKE_PREFIX_PATH
  • solve windows patch issue by removing C:/Strawberry/c/bin/patch.exe
  • replace outdated ilammy/msvc-dev-cmd@v1 by TheMrMilchmann/setup-msvc-dev@v4
  • free space on linux build by removing apt packages
  • use action test-reporter@v3 to create reports of ctest results
  • initial adapt build for codeql.yml

@misery see
https://github.com/chcg/AusweisApp/actions/runs/28314350091
https://github.com/chcg/AusweisApp/actions/runs/28314350095

@chcg
chcg force-pushed the gh_junit_test branch 3 times, most recently from a59b815 to 9f8d392 Compare June 24, 2026 21:34
@chcg
chcg force-pushed the gh_junit_test branch 3 times, most recently from 0d62058 to 7b5602b Compare June 28, 2026 06:53
@chcg
chcg force-pushed the gh_junit_test branch 4 times, most recently from ccb1a7f to cc9e911 Compare July 19, 2026 08:34
@misery

misery commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Thanks! Could you change all actions to pinned commit hash? So Require actions to be pinned to a full-length commit SHA option could be used?

@chcg

chcg commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

@misery Where do you see the advantage for this hardened usage of GH actions (see https://dev.to/suzukishunsuke/pin-github-actions-to-a-full-length-commit-sha-for-security-2n7p) as they are not running in your official repo ?
I could do it.

…brew for macos

- use env vars also for DCMAKE_PREFIX_PATH
- solve windows patch issue by removing C:/Strawberry/c/bin/patch.exe
- replace not maintained ilammy/msvc-dev-cmd@v1 by TheMrMilchmann/setup-msvc-dev@v4
- free space on linux build by removing apt packages
- use action test-reporter@v3 to create reports of ctest results
- adapt build for codeql.yml
chcg added a commit to chcg/AusweisApp that referenced this pull request Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants