Skip to content

Adding auto build#1

Merged
heathhenley merged 4 commits into
mainfrom
dev_add_action_to_build
Feb 11, 2026
Merged

Adding auto build#1
heathhenley merged 4 commits into
mainfrom
dev_add_action_to_build

Conversation

@heathhenley

@heathhenley heathhenley commented Feb 10, 2026

Copy link
Copy Markdown
Contributor

Add action to build, test, spit out the installable packages (dll/shared lib and headers) on windows and ubuntu

@heathhenley

Copy link
Copy Markdown
Contributor Author

These are here: https://github.com/farsounder/farsounder-cpp-client/actions/runs/21883409612

Can't believe it actually built first try - testing them each completely standalone to see if it's true.

@heathhenley
heathhenley marked this pull request as ready for review February 10, 2026 22:27
@heathhenley

Copy link
Copy Markdown
Contributor Author

These worked completely standalone 🎉 - linking just a basic_client.cpp against them on windows and ubuntu. Probably going to make that example available too as it's a lot simpler than what's in this repo.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a GitHub Actions workflow to automatically build, test, install (stage), and upload SDK package artifacts for Windows and Ubuntu, aligning CI with the project’s CMake install/export packaging approach.

Changes:

  • Added a CI workflow to build + run tests on Windows and Ubuntu and upload staged install artifacts.
  • Added validation checks to ensure the staged install contains expected binaries/headers/CMake package files/license.
  • Removed the completed TODO entry about adding GitHub Actions build/test from the README.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
README.md Removes the now-completed TODO item for GitHub Actions build/test.
.github/workflows/build-test-package.yml New workflow to configure/build/test/install/validate and upload packaged artifacts for Windows + Ubuntu.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/build-test-package.yml Outdated
Comment thread .github/workflows/build-test-package.yml Outdated
Comment thread .github/workflows/build-test-package.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@heathhenley
heathhenley requested a review from elap February 10, 2026 23:35

@elap elap left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Great to have this auto building!

@heathhenley
heathhenley merged commit c01636c into main Feb 11, 2026
2 checks passed
@heathhenley
heathhenley deleted the dev_add_action_to_build branch February 11, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants