CI: apply pre-commit format fix from #49806

[aaronsteers]

What

This PR applies the auto-format rules from:

• feat: pre-commit instead of airbyte-ci format #49806

How

Review guide

User Impact

Can this PR be safely reverted and rolled back?

• YES 💚
• NO ❌

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
airbyte-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 18, 2024 4:44am

[github-actions]

Format-fix job started... Check job output.

✅ Changes applied successfully. (c88f8a5)

[github-actions]

Format-fix job started... Check job output.

✅ Changes applied successfully. (d471a80)

[github-actions]

Format-fix job started... Check job output.

✅ Changes applied successfully. (0303939)

[natikgadzhi]

@aaronsteers the format-fix run complaints at:

The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-

Should we clean this up while we're here?

[aaronsteers]

@natikgadzhi -

@aaronsteers the format-fix run complaints at:

The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-

Should we clean this up while we're here?

It's been bugging me too. Created new PRs to clean these up.

• ci: replace deprecated ::set-output syntax #49856
• ci: use explicit ubuntu versioned ci runner #49857

[aaronsteers]

@natikgadzhi, @alafanechere - Failures appear to be related to version bumps:

Extra pair of eyes to confirm would be welcome.

https://github.com/airbytehq/airbyte/actions/runs/12386439202/job/34574311960?pr=49852

    def test_validation_pass_on_same_docker_image_tag(metadata_definition):
        success, error_message = metadata_validator.validate_docker_image_tag_is_not_decremented(metadata_definition, None)
>       assert success
E       assert False

Same error and URL and for all of these failures:

[sentry-io]

Suspect Issues

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

• ‼️ requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://tstdrv2253588.suitetalk.api.netsuite.com/services... /usr/local/lib/python3.9/site-packages/airbyte_... View Issue
• ‼️ requests.exceptions.HTTPError: 400 Client Error: for url: https://creator.zoho.com/api/v2/applications /usr/local/lib/python3.9/site-packages/airbyte_... View Issue

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