8000 [ci] clone with depth=1 to kill push-forced PRs by nicolas-grekas · Pull Request #17889 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[ci] clone with depth=1 to kill push-forced PRs #17889

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 1 commit into from
Feb 26, 2016

Conversation

nicolas-grekas
Copy link
Member
Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? no
Fixed tickets -
License MIT
Doc PR -

As documented in travis-ci/travis-ci#4575 (comment), setting a clone depth of 1 will make push-forced PRs to fail quickly so that our CI won't spend too much time on them.

@javiereguiluz
Copy link
Member

For anyone reading this pull request, keep in mind to not apply this change blindly in your own projects. GitHub employees recommend to use a reasonable git depth and Travis employees say that there is no noticeable performance improvement in the majority of cases.

@fabpot
Copy link
Member
fabpot commented Feb 26, 2016

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 562d4d6 into symfony:2.3 Feb 26, 2016
fabpot added a commit that referenced this pull request Feb 26, 2016
…-grekas)

This PR was merged into the 2.3 branch.

Discussion
----------

[ci] clone with depth=1 to kill push-forced PRs

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | no
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

As documented in travis-ci/travis-ci#4575 (comment), setting a clone depth of 1 will make push-forced PRs to fail quickly so that our CI won't spend too much time on them.

Commits
-------

562d4d6 [ci] clone with depth=1 to kill push-forced PRs
@nicolas-grekas nicolas-grekas deleted the depth-1 branch February 29, 2016 18:02
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.

4 participants
0