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

Include "summaryLength" config option #512

Closed
wants to merge 336 commits into from

Conversation

brennanangel
Copy link
Contributor

*Updated to include "summaryLength" configuration option from
gohugoio/hugo#3924
*Took line about HTML loading in summary and updated it to clarify the user has options to show html or not instead of implying there are no options
*Fixed typos

bep and others added 30 commits February 15, 2018 09:07
@kaushalmodi
Copy link
Contributor

@brennanangel Can you rebase this PR and force push to the PR branch? Thanks.

pintergreg and others added 5 commits September 12, 2018 12:07
*Updated to include "summaryLength" configuration option from gohugoio/hugo#3734
*Took line about HTML loading in summary and updated it to clarify the user has options to show html or not instead of implying there are no options
*Fixed typos
@kaushalmodi
Copy link
Contributor

@brennanangel That doesn't look good :) You need to rebase, not merge.

@kaushalmodi
Copy link
Contributor

@brennanangel If rebasing doesn't work out, you might just close this PR and create a new one, with the commit based off the latest master.

@brennanangel brennanangel deleted the patch-1 branch September 12, 2018 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.