Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

Pagespeed libraries #533

Closed
wants to merge 4 commits into from
Closed

Pagespeed libraries #533

wants to merge 4 commits into from

Conversation

vidluther
Copy link

Some documentation and a basic script to work with #532

Note, this won't work with WordPress sites yet.. those require some minor changes.

@igrigorik
Copy link
Contributor

👍

sudo cp ~/pagespeed_libraries.conf /etc/nginx/
```

In the Nginx.conf file above, right after you say pagespeed on; add this
Copy link
Contributor

Choose a reason for hiding this comment

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

pagespeed on; -> pagespeed on;

Copy link
Author

Choose a reason for hiding this comment

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

I'm not sure what you're asking/saying here :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, I just meant that I wanted "pagespeed on" to be surrounded in backticks so it would show up monospaced and look like a command. But I did this when I merged it, so no worries.

@jeffkaufman
Copy link
Contributor

Comments aside, looks good. Thanks for doing this!

Could you sign our CLA? https://developers.google.com/open-source/cla/individual

@jeffkaufman
Copy link
Contributor

@vidluther I'd like to merge this; could you sign the CLA?

@vidluther
Copy link
Author

Just signed it.. I need to make a fix too didn't I?

@jeffkaufman
Copy link
Contributor

@vidluther Thanks! I can make the fix.

jeffkaufman added a commit that referenced this pull request Oct 21, 2013
@jeffkaufman
Copy link
Contributor

Merged as 7c15cd8.

@jeffkaufman
Copy link
Contributor

Reworded the readme a bit: 687e0a7

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

Successfully merging this pull request may close these issues.

None yet

3 participants