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

tests: update chromium installable source path #12364

Merged
merged 2 commits into from
May 17, 2021
Merged

Conversation

paulirish
Copy link
Member

This is a minor followup from #11763. just a tweak to avoid a dumb 404.


also, spoiler alert: the issues panel test fails

image

but i (or someone else) will have to deal with that separately.

@paulirish paulirish requested a review from a team as a code owner April 14, 2021 19:21
@paulirish paulirish requested review from patrickhulce and removed request for a team April 14, 2021 19:21
@google-cla google-cla bot added the cla: yes label Apr 14, 2021
Copy link
Collaborator

@patrickhulce patrickhulce left a comment

Choose a reason for hiding this comment

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

does anyone get notified about these failures? I would expect to see some runs under "Markdown Links Checker" (oof name for these chromium checks), but I see 0 runs.

https://github.com/GoogleChrome/lighthouse/actions/workflows/cron-weekly.yml

@@ -16,7 +16,7 @@ describe('installabilityErrors', () => {

beforeAll(async () => {
const installableLoggingGitTilesUrl =
'https://chromium.googlesource.com/chromium/src/+/master/components/webapps/installable/installable_logging.cc?format=TEXT';
'https://chromium.googlesource.com/chromium/src/+/master/components/webapps/browser/installable/installable_logging.cc?format=TEXT';
Copy link
Collaborator

Choose a reason for hiding this comment

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

file moved?

Copy link
Member Author

Choose a reason for hiding this comment

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

ayup

@paulirish
Copy link
Member Author

does anyone get notified about these failures? I would expect to see some runs under "Markdown Links Checker" (oof name for these chromium checks), but I see 0 runs.

heh. the big changes to https://github.com/GoogleChrome/lighthouse/blob/master/.github/workflows/cron-weekly.yml just landed yesterday.. so its 'weekly' schedule hasnt been hit yet.

@paulirish
Copy link
Member Author

does anyone get notified about these failures?

tbh i'm not sure about notifications. i get some cron-ish gha notification on my other projects, but i'm not sure how thing will work with this.

they should def be visible in the 'Actions' tab after it runs tho.

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

Successfully merging this pull request may close these issues.

3 participants