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 `fixture_site` for Document tests #3511

Merged
merged 1 commit into from Feb 25, 2015

Conversation

Projects
None yet
3 participants
@alfredxing
Member

alfredxing commented Feb 25, 2015

Use the fixture_site helper in the Document tests instead of Site.new(Jekyll.configuration(overrides)). It turns out fixture_site already defaulted to full_rebuild: false

Should fix #3500

parkr added a commit that referenced this pull request Feb 25, 2015

Merge pull request #3511 from alfredxing/fix-docs-tests
Use `fixture_site` for Document tests

@parkr parkr merged commit 929d18f into jekyll:master Feb 25, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Feb 25, 2015

Member

🙌

Member

parkr commented Feb 25, 2015

🙌

parkr added a commit that referenced this pull request Feb 25, 2015

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

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