Skip to content

Fix #209, add number of successful subscriptions to initialisation event message - #210

Closed
thnkslprpt wants to merge 1 commit into
nasa:mainfrom
thnkslprpt:fix-209-add-number-of-successful-subscriptions-to-init-message
Closed

thnkslprpt wants to merge 1 commit into
nasa:mainfrom
thnkslprpt:fix-209-add-number-of-successful-subscriptions-to-init-message

Conversation

@thnkslprpt

Copy link
Copy Markdown
Contributor

Checklist

Describe the contribution

  • Fixes Add the number of successful TLM pipe subscriptions to the initialisation event message #209
    • adds the number of successful TLM pipe subscriptions to the initialisation event message string - this would be useful in general, and especially in cases where zero subscriptions were successful despite this not being the expectation
    • minor correction to TO_LAB_init return type
    • rest is formatting updates applied according to the cFS bundle's .clang-format settings (no longer enforced in the CI)

Testing performed
GitHub CI actions all passing successfully (incl. Build + Run, Unit/Functional Tests etc.).

Expected behavior changes
Additional useful information in initialisation message.

System(s) tested on
Debian 12 using the current main branch of cFS bundle.

Contributor Info
Avi Weiss   @thnkslprpt

@keegan-moore

Copy link
Copy Markdown
Contributor

We're working this internally alongside some other to_lab changes. Closing this as OBE.

@thnkslprpt
thnkslprpt deleted the fix-209-add-number-of-successful-subscriptions-to-init-message branch February 27, 2026 08:13
@chillfig chillfig added this to the v7.0.1 milestone May 13, 2026
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.

Add the number of successful TLM pipe subscriptions to the initialisation event message

3 participants