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

permalink: false option #19

Merged
merged 1 commit into from
Sep 2, 2014
Merged

Conversation

rahulkgupta
Copy link
Contributor

  • Pass permalink: false to a file to be ignored by the permalinks plugin

@aymericbeaumet
Copy link

Good job 👍. Will use your fork until it's merged.

@aymericbeaumet
Copy link

What do you think @ianstormtaylor ?

@ianstormtaylor
Copy link
Contributor

Ah interesting, happy to merge, think we can change the metadata key to .permalink === false instead?

@rahulkgupta
Copy link
Contributor Author

so would the header look like

permalink: false

?

@ianstormtaylor
Copy link
Contributor

Yup, exactly!

On Tue, Aug 26, 2014 at 5:29 PM, Rahul Gupta [email protected]
wrote:

so would the header look like

permalink: false

?


Reply to this email directly or view it on GitHub
#19 (comment)
.

@aymericbeaumet
Copy link

permalink: false is fine by me.

@rahulkgupta Please patch quickly :)

@rahulkgupta rahulkgupta changed the title permalinks-ignore option permalink: false option Aug 28, 2014
- Pass permalinks: false to a file to be ignored by the permalinks plugin
@rahulkgupta
Copy link
Contributor Author

updated.

@aymericbeaumet
Copy link

LGTM

@ianstormtaylor
Copy link
Contributor

sweetness

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.

3 participants