Skip to content

Add nightly tests workflow to test prior forks #8319

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 8 commits into
base: unstable
Choose a base branch
from
Open

Add nightly tests workflow to test prior forks #8319

wants to merge 8 commits into from

Conversation

@jimmygchen
Copy link
Member

@jimmygchen jimmygchen commented Oct 29, 2025 •
edited
Loading

Issue Addressed

Once #8271 is merged, CI will only cover tests for RECENT_FORKS (prev, current, next)

To make sure functionalities aren't broken for prior forks, we run tests for these forks nightly. They can also be manually triggered.

Tested via manual trigger here: https://github.com/jimmygchen/lighthouse/actions/runs/18896690117

image

@jimmygchen jimmygchen added test improvement Improve tests ready-for-review The code is ready for review skip-ci Don't run the `test-suite` labels Oct 29, 2025
@mergify
Copy link

mergify bot commented Oct 29, 2025

Some required checks have failed. Could you please take a look @jimmygchen? 🙏

@mergify mergify bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. and removed ready-for-review The code is ready for review labels Oct 29, 2025
@jimmygchen jimmygchen added ready-for-review The code is ready for review and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Oct 29, 2025
@mergify mergify bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. and removed ready-for-review The code is ready for review labels Oct 29, 2025
@jimmygchen jimmygchen added ready-for-review The code is ready for review and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Oct 29, 2025
@mergify

This comment was marked as outdated.

Sign in to view
@mergify mergify bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. and removed ready-for-review The code is ready for review labels Oct 29, 2025
This was referenced Oct 29, 2025
Merged
Open
@jimmygchen jimmygchen added ready-for-review The code is ready for review and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Oct 29, 2025
@mergify
Copy link

mergify bot commented Oct 29, 2025

Some required checks have failed. Could you please take a look @jimmygchen? 🙏

@mergify mergify bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. and removed ready-for-review The code is ready for review labels Oct 29, 2025
@jimmygchen jimmygchen removed the skip-ci Don't run the `test-suite` label Oct 29, 2025
@jimmygchen jimmygchen removed the waiting-on-author The reviewer has suggested changes and awaits thier implementation. label Oct 29, 2025
@jimmygchen jimmygchen added the ready-for-review The code is ready for review label Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

ready-for-review The code is ready for review test improvement Improve tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jimmygchen