Skip to content

Comments

GitHub Actions: Build and Push#892

Open
RobLoach wants to merge 4 commits intomasterfrom
build-and-push
Open

GitHub Actions: Build and Push#892
RobLoach wants to merge 4 commits intomasterfrom
build-and-push

Conversation

@RobLoach
Copy link
Contributor

@RobLoach RobLoach commented Feb 7, 2026

This adds a CI workflow to paq.sh and push the changes to master.

@RobLoach RobLoach changed the title Add build and deploy GitHub Actions: Build and Push Feb 8, 2026
@PavelSharp
Copy link
Contributor

PavelSharp commented Feb 18, 2026

It seems this PR enables automatic building of Nuklear.h. To prevent users from accidentally modifying this generated file, maybe we should consider adding it to .gitignore? In that case, we would need to use git add -f in the workflow to commit it.
To my knowledge, GitHub Actions Workflows run in parallel. This means that ccpp.yml could fail while build-and-push.yml completes successfully.

@RobLoach
Copy link
Contributor Author

It seems this PR enables automatic building of Nuklear.h. To prevent users from accidentally modifying this generated file, maybe we should consider adding it to .gitignore?

While this seems sound, many people just git clone nuklear, and then refernecee nuklear.h directly. Wouldn't want to break their workflow.

To my knowledge, GitHub Actions Workflows run in parallel. This means that ccpp.yml could fail while build-and-push.yml completes successfully.

Good call. We'll have to make sure this runs after the tests.

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