Use with page speed insights

[kaihendry]

Could you please illustrate how you would use https://github.com/nwtgck/actions-netlify#outputs with https://github.com/marketplace/actions/page-speed-insights ?

Thank you!

[nwtgck]

@kaihendry Hi,

Here is the yaml.

...
      - name: Running Page Speed Insights
        uses: jakepartusch/psi-action@v1
        with:
          url: ${{ steps.deploy-to-netlify.outputs.deploy-url }}
          threshold: 70
          strategy: mobile

full: https://github.com/nwtgck/actions-netlify/blob/85450f952f626b64861a3facdb5d19519fea07f8/.github/workflows/test.yml

Here is the actual changes: 85450f9 #85

Result

Actions: https://github.com/nwtgck/actions-netlify/pull/85/checks?check_run_id=564141914

[kaihendry]

Didn't work on kaihendry/dabase.com#10 ... any ideas why?

Something obviously with the URL https://github.com/kaihendry/dabase.com/pull/10/checks?check_run_id=566227674

Maybe the name is wrong? https://github.com/kaihendry/dabase.com/blob/master/.github/workflows/netlify.yml

[kaihendry]

Would prefer to see Pagespeed output on the PR itself...

[nwtgck]

I'm not sure why the error caused. But it says " ##[error]Url is required to run Page Speed Insights."

[nwtgck]

Have you tried the following setting? Because the error might not cause by actions-netlify.

...
        with:
          url: "https://dabase.com/"
          threshold: 70
          strategy: mobile

[nwtgck]

@kaihendry Have you tried this?

[AllanOricil]

I think it had an error because it was a draft. To run it you would need the draft to be published.

[nwtgck]

@kaihendry ping?? I'd like to close the issue when you solved this.