airbyte-ci: re-introduce mypy

[alafanechere]

What

Re-introduce changes from #33785 which got reverted due to a regression on test and publish pipeline:

• Publish pipelines raised an assertion error
• Connector test pipelines were not setting the test report url correctly on commit status

How

• Revert airbyte-ci: Revert mypy changes #33954
• Fix regressions:
  • The assert report is not None was set too early in the code path
  • Setting report URL on commits was removed by my changes in airbyte-ci: fix types (mypy) #33785

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
airbyte-docs	⬜️ Ignored (Inspect)	Visit Preview		Jan 5, 2024 10:14am

[alafanechere]

Current dependencies on/for this PR:

• master
  • PR airbyte-ci: re-introduce mypy #33964 👈

This stack of pull requests is managed by Graphite.

[alafanechere]

/approve-and-merge reason="fixing airbyte-ci regression"

[octavia-approvington]

All in!!

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ TransportError: Unexpected response from engine: Server error '502 Bad Gateway' for url 'http://127.0.0.1:33345/q... dagger.client._core in execute View Issue
• ‼️ SessionError: Failed to start Dagger engine session: Command '/home/runner/.cache/dagger/dagger-0.9.5 session -... dagger._engine.session in get_connect_params View Issue
• ‼️ AttributeError: 'PosixPath' object has no attribute 'endswith' pipelines.models.contexts.pipeline_context in g... View Issue

_{Did you find this useful? React with a 👍 or 👎}