Skip to content

Fix CI for external contributions #68

Merged
antejavor merged 14 commits intomasterfrom
fix-ci-cd-action
Aug 14, 2025
Merged

Fix CI for external contributions #68
antejavor merged 14 commits intomasterfrom
fix-ci-cd-action

Conversation

@antejavor
Copy link
Copy Markdown
Collaborator

@antejavor antejavor commented Aug 12, 2025

This PR fixes so the CI is being run on the external contribution PRs.

The windows fix uses vcpkg and statically linked OpenSSL

Fixes:
#69

@antejavor antejavor marked this pull request as ready for review August 12, 2025 09:50
@antejavor antejavor requested a review from mattkjames7 August 12, 2025 09:51
@antejavor antejavor self-assigned this Aug 12, 2025
@antejavor antejavor requested a review from gitbuda August 12, 2025 09:52
@antejavor
Copy link
Copy Markdown
Collaborator Author

@gitbuda So I want to run the CI tests on external contributor PRs, needed to change the CI a bit.

I would like you to give me more rights on this repo so I can configure that CI needs approval. 🙇

@antejavor antejavor changed the title Fix CI/CD for external contributions Fix CI for external contributions Aug 12, 2025
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
@antejavor antejavor requested a review from mattkjames7 August 14, 2025 11:22
@antejavor antejavor linked an issue Aug 14, 2025 that may be closed by this pull request
Copy link
Copy Markdown

@mattkjames7 mattkjames7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@antejavor antejavor merged commit 80933ab into master Aug 14, 2025
5 checks passed
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.

Fix Windows build and test

2 participants