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.

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

Replace check-stability script with wpt run --stability and document how to run locally #10269

Closed
Ms2ger opened this issue Apr 2, 2018 · 6 comments
Closed

Replace check-stability script with wpt run --stability and document how to run locally #10269

Ms2ger opened this issue Apr 2, 2018 · 6 comments
Assignees
@gsnedders
Labels
ci_stability docs infra priority:roadmap

Comments

@Ms2ger
Copy link
Contributor

Ms2ger commented Apr 2, 2018

I'm not even sure it's intended to use locally, but it seems to assume quite heavily that it's running on TravisCI. For local use, a way to run it with a list of tests rather than revisions might be nice?

CC @foolip
@jgraham
Copy link
Contributor

jgraham commented Apr 2, 2018

wpt run --stability does something mostly equivalent. Ideally the travis job would change to use that, but there are quite a few details to get right around reporting stuff correctly.

@mrego
Copy link
Member

mrego commented Apr 2, 2018

JFYI, both run --stability and check-stability run fine for me locally.

@foolip
Copy link
Member

foolip commented Apr 2, 2018

Is there anything that needs to be done here? Just documentation?

@gsnedders
Copy link
Member

Documentation and making CI use wpt run --stability.

@foolip foolip added the docs label Apr 2, 2018
@foolip foolip changed the title Review if we can make check-stability easier to use locally Replace check-stability script with wpt run --stability and document how to run locally Apr 2, 2018
@foolip foolip mentioned this issue Apr 3, 2018
Closed
@foolip foolip mentioned this issue May 8, 2018
Closed
@foolip foolip assigned gsnedders and unassigned kereliuk May 8, 2018
@gsnedders
Copy link
Member

Per #10921 --stability is just a subset of behaviour of --verify and should really cease being separate, so let's close this in favour of #9874.

@gsnedders
Copy link
Member

Also starting to look into this, we already use --stability.

@foolip foolip mentioned this issue Oct 7, 2018
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gsnedders gsnedders

Labels
ci_stability docs infra priority:roadmap
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@mrego @Ms2ger @gsnedders @jgraham @foolip @kereliuk and others