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 auto-generate stamp to certain pages. #345

Merged
merged 3 commits into from Mar 23, 2017
Merged

Add auto-generate stamp to certain pages. #345

merged 3 commits into from Mar 23, 2017

Conversation

jpmedley
Copy link
Contributor

@addyosmani
Copy link
Member

I've re-run the build and this appears to be failing on Travis atm.

@@ -29,6 +29,8 @@ navigation_weight: 0

`;

const AUTO_GENERATED = "<!-- DO NOT EDIT. This page is autogenerated. -->";

Choose a reason for hiding this comment

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

Lint is throwing an error on this because it's double quote, change to single quote to work.

Test locally by running gulp lint.

@jpmedley
Copy link
Contributor Author

@jeffposnick @addyosmani @gauntface PTAL

Copy link

@gauntface gauntface left a comment

Choose a reason for hiding this comment

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

Might be worth adding info on gulp document and / or the templates directory - just to direct peoples attention.

@jpmedley jpmedley merged commit 3a0bd5c into master Mar 23, 2017
@jpmedley jpmedley deleted the stamp branch March 23, 2017 19:21
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.

None yet

3 participants