Merged
Conversation
added 5 commits
August 14, 2025 11:19
There was a problem hiding this comment.
Pull Request Overview
This pull request implements CI/CD infrastructure for the TrySpace Lab project, including GitHub Actions workflows and significant refactoring of the build system. The changes focus on standardizing Docker image naming, improving the Makefile structure, and adding automated builds for key components.
- Updates submodule references to latest commits
- Refactors Makefile with improved organization and new targets
- Adds GitHub Actions CI workflow for automated building
- Standardizes Docker image naming from
tryspace-labtotryspaceorg/tryspace-lab
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| simulith | Updates submodule commit reference |
| gsw | Updates submodule commit reference |
| fsw | Updates submodule commit reference |
| comp/demo | Updates submodule commit reference |
| cfg/Dockerfile.base | Removes version pinning from packages and improves pip installation |
| Makefile | Major refactoring with simplified targets and improved organization |
| .github/workflows/ci.yml | Adds CI workflow for automated building of simulith and FSW components |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
jlucas9
pushed a commit
that referenced
this pull request
Aug 18, 2025
* [#1] Added tryspace-fsw as a submodule; * [#1] Updated fsw after confirmed successfully building and creating runtime; * [#1] Updated FSW submodule after PR review; * [#5] Added demo component as a submodule; * [#5] Added simultih submodule; * [#5] Moved to use a tryspace-lab container as the base; * [#5] Demo component cli and sim building; * [#5] Demo component CLI to SIM getting time tics; * [#5] support/cli-compose.yml confirmed functional; * [#5] Made building simulith tests optional and added CLI interface for server to pause/play and run faster/slower than real time; * [#5] Submodule updates after testing and merges; * [#3] Created lab-compose.yml file and updated FSW to use simulith time; * [#3] Launch demo sim in lab compose; * [#3] Updates after submodule merges; * [#6] Created director of simulations; * [#6] Updates to FSW an simulith to improve OSAL; * [#6] Change to use local IPC for simulith ZMQ; * [#6] Enable demo app by default; * [#6] Submodule updates after merges; * [#11] Add 42 to simulith director and SVB use in demo component; * [#11] Merge submodules after PR review; * [#7] Move cli-compose.yml up to top level tryspace-lab; * [#7] Updates to hwlib to resolve FSW use; * [#7] Initial tryspace-orchestrator.py with missions and scenarios; * [#7] Additional updates to further clean up and cover for a lack of overrides; * [#7] Submodule updates after merges; * [tryspaceorg/tryspace#38] Remove previous license.txt; * [tryspaceorg/tryspace#38] Added AGPLv3 LICENSE.txt; * [#38] Updated submodules with AGPLv3; * [#38] Added GSW submodule; * [#23] Working with cFS CI/TO_Lab; * [#23] Updates to submodules after PR review; * [#15] Initial attempt at templates; * [#13] Created CONTRIBUTING.md; * [#12] Added atlas with documentation; * [#40] Moved simulith calls to PSP; * [#40] Functional now with SCH; * [#40] Updates after PR reviews and merges; * [#14] CI (#45) * [#14] Cleanup of Dockerfile.base and Makefile, initial CI.yml; * [#14] Updates to use public container and cleanup make/cmake; * [#14] Use public container in ci.yml; * [#14] Attempt to build component sims and added orchestrator step to FSW CI; * [#14] Updated ci.yml formatting; * [#14] Added cfg step to build-simulith; * [#14] Updates after submodule merges; * [#21] Updates for FSW unit tests; (#46) * [#21] Updates for FSW unit tests; * [#21] Added print on error to ci.yml for unit tests; * [#21] Ignored failing OSAL tests for now - revisit these! * [#21] Ignored additional OSAL tests due to CI failures, added PR template for release; * [#21] Added curl and gpg to base container for codecov; * [#21] Submodule updates after merges * [#47] Release 0.0.0 to `dev` (#49) * [#47] User guide updates after review; * [#47] Change to versioned container; * [#47] Limited number of parallel jobs and removed env; * [#47] PR suggestions and removed compose restart; * [#47] Submodule merges; * [#47] Update to PR release template;
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #14
Submodule PRs: