Skip to content

Comments

airbyte-to-flow: strip trailing newlines when parsing run_interval_minutes.json#351

Merged
Alex-Bair merged 1 commit intomasterfrom
bair/airbyte-to-flow-strip-newlines-from-run-interval
Jan 28, 2026
Merged

airbyte-to-flow: strip trailing newlines when parsing run_interval_minutes.json#351
Alex-Bair merged 1 commit intomasterfrom
bair/airbyte-to-flow-strip-newlines-from-run-interval

Conversation

@Alex-Bair
Copy link
Member

The run_interval_minutes.json files in some connectors (source-youtube-analytics, source-exchange-rates, source-google-analytics-data-api) have trailing newlines, causing parse::<u64>() to fail with "invalid digit found in string".

Adding .trim() before parsing handles any whitespace in these config files.

…nutes.json

The `run_interval_minutes.json` files in some connectors (`source-youtube-analytics`,
`source-exchange-rates`, `source-google-analytics-data-api`) have trailing newlines,
causing `parse::<u64>()` to fail with "invalid digit found in string".

Adding `.trim()` before parsing handles any whitespace in these config files.
@Alex-Bair Alex-Bair requested a review from a team January 28, 2026 15:12
@Alex-Bair Alex-Bair merged commit 625fba4 into master Jan 28, 2026
45 of 57 checks passed
@Alex-Bair Alex-Bair deleted the bair/airbyte-to-flow-strip-newlines-from-run-interval branch January 28, 2026 15:45
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