-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chore(ci): Add version of format-checks that can run on forks #37356
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Closing in favor of We can re-open this if we decide to go a different direction. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with caveat that Augustin then cleans up the "consistency" piece when run-airbyte-ci is ready to run with optional tokens. Let's merge this!
Note: this PR is deprioritized in favor of:Brought back as preferred over:
Since the new version is so much faster than the old version, I make a case in the above that we should just always run the unprivileged version.
This just adds a simple alternate execution of the
format-check
job. The alternate version will run on forks. The existing version, which requires 8-10 distinct secrets, will be skipped on forks.For security/consistency reasons, the new check does not use airbyte-ci from the community member's local repo - instead installing from our own master branch (ignoring
airbyte-ci
from the forked repo).