Skip to content

Workflow that runs all tests#144

Merged
elenagaljak-db merged 8 commits intomainfrom
elenagaljak-db-all_test
Mar 18, 2026
Merged

Workflow that runs all tests#144
elenagaljak-db merged 8 commits intomainfrom
elenagaljak-db-all_test

Conversation

@elenagaljak-db
Copy link
Contributor

@elenagaljak-db elenagaljak-db commented Mar 16, 2026

What changes are proposed in this pull request?

When we change Rust SDK (which is the base of all SDKs) we want to run all tests, not only rust related ones.

How is this tested?

Manually

@elenagaljak-db elenagaljak-db marked this pull request as ready for review March 16, 2026 15:12
@elenagaljak-db elenagaljak-db force-pushed the elenagaljak-db-all_test branch from c6afaeb to 0a92ebc Compare March 17, 2026 13:00
Copy link
Contributor

@teodordelibasic-db teodordelibasic-db left a comment

Choose a reason for hiding this comment

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

Looks good except the two testing lines 🙂

Ok(stream) => {
if let Some(stream_id) = stream.stream_id.as_ref() {
info!(stream_id = %stream_id, "Successfully created new ephemeral stream");
info!(stream_id = %stream_id, "Successfully created new ephemeral stream. random line change");
Copy link
Contributor

Choose a reason for hiding this comment

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

Oops.


async def main():
print("Starting asynchronous ingestion example (Protobuf)...")
print("Starting asynchronous ingestion example (Protobuf)... random line change")
Copy link
Contributor

Choose a reason for hiding this comment

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

Oops.

Signed-off-by: elenagaljak-db <elena.galjak@databricks.com>
Signed-off-by: elenagaljak-db <elena.galjak@databricks.com>
Signed-off-by: elenagaljak-db <elena.galjak@databricks.com>
Signed-off-by: elenagaljak-db <elena.galjak@databricks.com>
Signed-off-by: elenagaljak-db <elena.galjak@databricks.com>
Signed-off-by: elenagaljak-db <elena.galjak@databricks.com>
Signed-off-by: elenagaljak-db <elena.galjak@databricks.com>
@elenagaljak-db elenagaljak-db force-pushed the elenagaljak-db-all_test branch from e384806 to 1bd97ab Compare March 17, 2026 15:07
@elenagaljak-db elenagaljak-db added this pull request to the merge queue Mar 18, 2026
Merged via the queue into main with commit a700851 Mar 18, 2026
69 checks passed
@elenagaljak-db elenagaljak-db deleted the elenagaljak-db-all_test branch March 18, 2026 11:11
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