Skip to content

Conversation

@cnunciato
Copy link
Contributor

Accounts for the shift into /docs.

@cnunciato cnunciato requested a review from justinvp June 7, 2019 20:53
Copy link
Member

@justinvp justinvp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM otherwise

Makefile Outdated
hugo
node ./scripts/build-search-index.js < ./public/search-data/index.json > ./public/search-index.json
node ./scripts/build-search-index.js < ./public/docs/search-data/index.json > ./public/docs/search-index.json
rm -rf ./public/search-data
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
rm -rf ./public/search-data
rm -rf ./public/docs/search-data

@@ -0,0 +1,10 @@
{
{{- $pages := (where .Site.Pages ".Params.noindex" "ne" true) -}}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete layouts/_default/search-data.json so this shows up in git as a move?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, totally thought I had. Good catch, thanks.

@@ -0,0 +1,7 @@
{{ define "main" }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete layouts/_default/search.html so this shows up in git as a move?

Signed-off-by: Christian Nunciato <c@nunciato.org>
@cnunciato cnunciato merged commit 5d9f14f into fusion Jun 7, 2019
@pulumi-bot pulumi-bot deleted the cnunciato/search branch June 7, 2019 22:13
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