Skip to content
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

Use shallow clone in the getting started example #22407

Merged
merged 1 commit into from
Feb 7, 2023

Conversation

kfirgollan
Copy link
Contributor

What

The AirByte repository got pretty big. This results in rather long clone time for getting started, which is mostly redundant. By using the --depth option the clone becomes much faster.

How

Minor update in the main README file.
Add --depth 1 for the clone example.

🚨 User Impact 🚨

In case of users without experience with git changing branches can be harder after clone.

The AirByte repository got pretty big. This results in rather long clone time for getting started, which is mostly redundant.
By using the `--depth` option the clone becomes much faster. I find it to be a better way to get started with the project quickly.
@CLAassistant
Copy link

CLAassistant commented Feb 6, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@juweins juweins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @kfirgollan ! Thank you for this addition to increase user experience! Sometimes, there is only little change needed for improvement!

Welcome to the Contributor of Airbyte! 🎉

Regarding this PR: LGTM!

Does it make sense to add a note clarifying why we use depth 1? Or does that speak for itself (From a new/juniors' perspective)

@sajarin it's your turn

@marcosmarxm marcosmarxm merged commit b41267f into airbytehq:master Feb 7, 2023
danidelvalle pushed a commit to danidelvalle/airbyte that referenced this pull request Feb 9, 2023
The AirByte repository got pretty big. This results in rather long clone time for getting started, which is mostly redundant.
By using the `--depth` option the clone becomes much faster. I find it to be a better way to get started with the project quickly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants