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

Not compatible with new psi version #3

Open
celinekurpershoek opened this issue Jan 13, 2015 · 3 comments
Open

Not compatible with new psi version #3

celinekurpershoek opened this issue Jan 13, 2015 · 3 comments

Comments

@celinekurpershoek
Copy link

Is it possible that this example is not compatible with the new psi version?

I get an string expected error. The way i could solve the problem is to use the following code in my gulp task:

psi.output(site, {
nokey: 'true', // or use key: ‘YOUR_API_KEY’
strategy: 'mobile'
}, cb);

psi version: 1.0.2

@addyosmani
Copy link
Owner

Confirmed. The API changed. I'll get this updated.

@addyosmani
Copy link
Owner

Master should be working now. Could you retry?

@celinekurpershoek
Copy link
Author

It's working again thnx! Only the output in the terminal is an object.
If i use the psi.ouput() it generates a nice report.

I also combined this project with gnork, this way i can test my local environment. And its working nice together ;)

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

No branches or pull requests

2 participants