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

Add support for includes.d on all sites #966

Merged
merged 1 commit into from
Mar 17, 2018

Conversation

partounian
Copy link
Contributor

No description provided.

@swalkinshaw
Copy link
Member

This works I assume? 😄 I kind of assumed it might not and only files within a site name subdir would be copied over but looking at the code this might work.

What about just putting these in the root include.d dir? And not all?

@partounian
Copy link
Contributor Author

partounian commented Mar 14, 2018 via email

@swalkinshaw
Copy link
Member

I'm fine either way. @fullyint does this make sense to you?

@fullyint
Copy link
Contributor

👍 Looks good to me. I lean toward keeping the all directory. Thanks @partounian!

@swalkinshaw swalkinshaw merged commit 447adc3 into roots:master Mar 17, 2018
@swalkinshaw
Copy link
Member

@partounian would you like to add a note to our docs about this?

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