unrevert #32806

[postamar]

Reverts the revert of #32806 and fixes the following:

• stop relying on git_revision when computing a diff because it's not always what we think it is (when git_branch is not master, it's an orphan merge commit)
• stop eagerly computing the diff

I have no idea how to test any of this. This is as good as I can do.

[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		Nov 28, 2023 4:04pm

[bnchrch]

Best we could do to test is to set the --is-ci flag locally and run using modified a few times.

Thinking about it the worst case scenario here is we merge, do a prepublish test using modified on a subsequent branch, and if that goes bad, revert again.

If thats true then Im personally fine to merge and play with fire on master

[postamar]

Thanks for taking a look.

Best we could do to test is to set the --is-ci flag locally and run using modified a few times.

That's as far as I got. OK, I'll merge this tomorrow morning early. Let's see what gives!

[sentry-io]

Suspect Issues

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

• ‼️ ExecError: failed to solve: process "git remote add --fetch --track augustin/12-8/fix-airbyte-ci-doc --track... dagger.api.base in _handle_execute View Issue

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