Skip to content

Adds the example/cpp-exchange/redis example to the test suite. - #41

Merged
MattToast merged 1 commit into
mainfrom
redis-example-updates
Aug 26, 2026
Merged

Adds the example/cpp-exchange/redis example to the test suite.#41
MattToast merged 1 commit into
mainfrom
redis-example-updates

Conversation

@MattToast

@MattToast MattToast commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Adds the example/cpp-exchange/redis example to the test suite.

However, this example will be skipped in the CI unless/until we decide to install SmartSim and SmartRedis prior to running the test suite. This is simply a convenience for people who have the example set up to check that the output matches what is expected through the test suite.

However, this example will be skipped in the CI unless/until we decide
to install SmartSim and SmartRedis prior to running the test suite. This
is simply a convinece for people who have the example set up to check
that the output matches what is expected through the test suite.
@MattToast

Copy link
Copy Markdown
Collaborator Author

Merging this without review as it is:

  1. Needed for fast approaching workshops
  2. Not being directly tested in our CI

Any review comments can come as part of a PR that addresses #25

@MattToast
MattToast merged commit 87cfff7 into main Aug 26, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant