Skip to content

Add --fast flag to utils/update-checkout #1264

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

Merged
merged 2 commits into from
Feb 16, 2016

Conversation

takebayashi
Copy link
Contributor

To build snapshots, it is not necessary to fetch all history.
--clone --fast flags make utils/update-checkout fetch latest revisions only.

@gribozavr
Copy link
Contributor

I think this is a good addition, but 'fast' does not describe what the flag does. "Clone sources in a fast way" -- "of course I want my sources fast, why isn't this the default".

Can you think of a better way to name the flag so that the implications are clear?

@dduan
Copy link
Contributor

dduan commented Feb 11, 2016

It's safe to assume users of this script understand what --depth=1 means (or at least can guess/google it), I would include it in the help text.

@gribozavr beat me to it. The flag name should be clearer. My $0.02:

  • --shallow
  • --skip-history

@takebayashi
Copy link
Contributor Author

renamed to --skip-history

gribozavr added a commit that referenced this pull request Feb 16, 2016
Add --fast flag to utils/update-checkout
@gribozavr gribozavr merged commit e5848cc into swiftlang:master Feb 16, 2016
@gribozavr
Copy link
Contributor

@takebayashi Thank you!

MaxDesiatov pushed a commit that referenced this pull request Oct 19, 2020
[pull] swiftwasm from master
@AnthonyLatsis AnthonyLatsis added the update-checkout Area → utils: the `update-checkout` script label Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update-checkout Area → utils: the `update-checkout` script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants