Ella Rohm-Ensing

Results 124 comments of Ella Rohm-Ensing

> [!WARNING] > This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite....

@bnchrch I see that they said it will probably fix this issue (even though we don't know how it was introduced in the first place) because of the issue of...

* **#38247** 👈 * `master` This stack of pull requests is managed by Graphite. Learn more about stacking. Join @erohmensing and the rest of your teammates on Graphite

* **#38246** 👈 * `master` This stack of pull requests is managed by Graphite. Learn more about stacking. Join @erohmensing and the rest of your teammates on Graphite

> [!WARNING] > This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite....

@alafanechere thanks! this actually clears things up for me - we're only linting _our_ code, not the whole codebase. So if we wanted to e.g. lint connector code, we would...

Grooming: Try hardcoding defaults to a minute We could potentially later follow up with moving build reports to the domain Conor gave us for the metadata project.

There is also custom handling for 403s in the [should_retry](https://github.com/airbytehq/airbyte/blob/991789976eb6b0da9b997e7a2b52caad319261e3/airbyte-integrations/connectors/source-freshdesk/source_freshdesk/streams.py#L59-L64) method - I think we can remove that now! We want to look for these and remove them as we...

FWIW, test commands have been working fine for me today and yesterday 🤔 @evantahler where did this run come from?

Ah right, okay. Thanks! Ignore me lol