-
Notifications
You must be signed in to change notification settings - Fork 52
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
Allow to upload results to LH CI server #20
Comments
Hey! I think that'd be great but I would suggest offering a way to "merge" the configs instead. Our lighthouserc.yml right now is pretty barebones. We generate it for the following reason:
But we definitely are lacking when it comes to "Make the important things easy, everything else possible." The everything else possible part we overlooked 🙃 What would you think of offering an option to merge the configs somehow (I think the one in here should be the "default" and the one a user provide would be overrides)? Is that something you'd be interested in doing? |
Hey! I just encountered another issue with this action and then remembered about this issue :) Would it be ok to use name: Shopify Lighthouse CI
on: [push]
jobs:
lhci:
name: Lighthouse
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: Lighthouse
uses: shopify/lighthouse-ci-action@1.0
with:
app_id: ${{ secrets.SHOP_APP_ID }}
app_password: ${{ secrets.SHOP_APP_PASSWORD }}
store: ${{ secrets.SHOP_STORE }}
custom_config:
ci:
upload:
target: 'lhci'
serverBaseUrl: 'https://your-lhci-server-url.example.com'
token: 'Your *build token* goes here' The default
Would something like this work? |
if you allow these ENV variables part of the shell script, we will able to send data to custom lighthouse server
|
Hey!
I've just set up this action for our repo and it works great. Thanks!
It would be great to be able to track how these metrics change over time, so I started looking into LH CI server. Would it be enough to allow one to set LH CI server URL and token and then change the generated
lighthouserc.yml
to include this config:if LH CI server URL and token are provided? If so, I could prepare a PR.
The text was updated successfully, but these errors were encountered: