Skip to content

ci: separate CI and test report workflows#76

Merged
leif81 merged 1 commit intoopen-dis:masterfrom
ngjunsiang:master
Sep 23, 2025
Merged

ci: separate CI and test report workflows#76
leif81 merged 1 commit intoopen-dis:masterfrom
ngjunsiang:master

Conversation

@ngjunsiang
Copy link
Contributor

@ngjunsiang ngjunsiang commented Sep 20, 2025

This PR separates the build_test.yml workflow into two files as recommended in the GitHub Action documentation, resolving a bug that causes CI to fail at the "Upload Test Results" step for PRs from forked repositories.

Fix #75

…coverage reporting

This resolves a bug that causes CI unit tests to fail when run from forked repositories without write permission.
@leif81 leif81 merged commit d061f1b into open-dis:master Sep 23, 2025
2 checks passed
@leif81
Copy link
Member

leif81 commented Sep 23, 2025

That's great. Thank-you @ngjunsiang

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.

CI unit tests fail in forked repositories

2 participants