Skip to content
This repository was archived by the owner on Aug 10, 2022. It is now read-only.

Makes building RSS/ATOM feeds optional #6225

Merged
merged 1 commit into from
Jun 5, 2018
Merged

Makes building RSS/ATOM feeds optional #6225

merged 1 commit into from
Jun 5, 2018

Conversation

petele
Copy link
Contributor

@petele petele commented Jun 5, 2018

Makes building of RSS/ATOM feeds optional. Takes general build time from about 10 seconds to about 3 seconds.

OK to review, but DO NOT MERGE until I've landed to corresponding CL

@petele petele requested a review from ebidel June 5, 2018 19:27
@WebFundBot
Copy link

👍

@@ -169,6 +170,7 @@ gulp.task('build:showcase', function() {
});

// Generate the RSS & ATOM feeds
options.title = 'Show Cases';
Copy link
Contributor

Choose a reason for hiding this comment

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

one word?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

All other instances in here are two words.

@petele petele merged commit 5954d29 into master Jun 5, 2018
@petele petele deleted the rss-on-demand branch June 5, 2018 20:17
@WebFundBot
Copy link

🎉 This has been pushed live to https://developers.google.com/web/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants