-
Notifications
You must be signed in to change notification settings - Fork 127
Proposal: Add link to the report in the bot comment #11
Comments
Good call. Want to send a PR? |
@ebidel btw, what's the retention policy for these reports ? how long are they going to be available ? |
There's no guarantees on uptime, retention, or reports being available, long-term. I suggest running your own setup if that's something important to you :) |
Is there any progress on this? |
@ebidel I'm interested in making a PR for this, but I'd love some insights 🙂 So we can generate the html file by passing
One thing we could do is add a flag in the request to This can be done in lighthousebot/builder/server.js Line 41 in 7f87bca
|
Hey @robinmetral @balazsorban44 @abdonrd ✋ lighthouse-keeper uploads not only full html reports but also keeps track of how metrics change between PR. You should check it out! |
@krzkaczor thanks, I'll take a look! 😃 |
It would be great if the bot comment had a link to the report.
From:
To:
The text was updated successfully, but these errors were encountered: