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

CI(Appveyor): shallow clone with 5 last commits #7312

Merged
merged 1 commit into from Oct 10, 2018

Conversation

Projects
None yet
5 participants
@XhmikosR
Contributor

XhmikosR commented Oct 10, 2018

No description provided.

Update appveyor.yml
Reduce git depth and reorder a few sections

@XhmikosR XhmikosR referenced this pull request Oct 10, 2018

Merged

Initialize AppVeyor CI to test plugin on Windows #77

2 of 2 tasks complete
@parkr

This comment has been minimized.

Member

parkr commented Oct 10, 2018

@XhmikosR Can you explain what you're doing here? The only substantive change I see is clone moved from 10 to 5.

@XhmikosR

This comment has been minimized.

Contributor

XhmikosR commented Oct 10, 2018

Yup, nothing else. The rest is a cosmetic change, since environments makes more sense to be first to me :)

@parkr

parkr approved these changes Oct 10, 2018

@parkr

This comment has been minimized.

Member

parkr commented Oct 10, 2018

Ok, works for me!

ashmaroli added a commit to ashmaroli/jekyll-watch that referenced this pull request Oct 10, 2018

@XhmikosR

This comment has been minimized.

Contributor

XhmikosR commented Oct 10, 2018

BTW @parkr I've enabled caching on AppVeyor for PR too, assuming no file that we've set is changed. Should speeds things up a bit in the future.

@DirtyF DirtyF changed the title from Update appveyor.yml to CI(Appveyor): shallow clone with 5 last commits Oct 10, 2018

@ashmaroli

👍

@DirtyF

This comment has been minimized.

Member

DirtyF commented Oct 10, 2018

@jekyllbot: merge +dev

@jekyllbot jekyllbot merged commit f6f6a56 into jekyll:master Oct 10, 2018

3 of 4 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
WIP ready for review
Details
codeclimate All good!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@DirtyF DirtyF removed the fix label Oct 10, 2018

@XhmikosR XhmikosR deleted the XhmikosR:patch-1 branch Oct 11, 2018

@jekyll jekyll deleted a comment from tedmelvin25 Oct 16, 2018

@jekyll jekyll locked as resolved and limited conversation to collaborators Oct 16, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.