Summary

Make the (interactive) output of run-install-tests much nicer.

Review Request #14876 — Created March 5, 2026 and updated March 5, 2026, 3:45 p.m.

Information

Owner

david

Repository

rbinstall

Branch

master

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

Since we have the rich library available in the rbinstall context, we
might as well use it. This makes it so parallel builds display in a live
table.

This also adds a --list argument which will print a list of the
available build targets (in case we want to do something like add this
to CI later).

Testing Done

Ran install tests and saw much nicer output.

Commits

Summary	ID
Make the (interactive) output of run-install-tests much nicer. Since we have the `rich` library available in the rbinstall context, we might as well use it. This makes it so parallel builds display in a live table. This also adds a `--list` argument which will print a list of the available build targets (in case we want to do something like add this to CI later). Testing Done: Ran install tests and saw much nicer output.	wvypxystvwoszxtpxwuzwulylpptnyzr

Summary

Make the (interactive) output of run-install-tests much nicer.

Since we have the `rich` library available in the rbinstall context, we might as well use it. This makes it so parallel builds display in a live table. This also adds a `--list` argument which will print a list of the available build targets (in case we want to do something like add this to CI later). Testing Done: Ran install tests and saw much nicer output.

wvypxystvwoszxtpxwuzwulylpptnyzr

Files

Issues

Description	From	Last Updated
line too long (83 > 79 characters) Column: 80 Error code: E501	reviewbot	March 5, 2026, 2:59 p.m.
Blank line here.	chipx86	March 5, 2026, 3:33 p.m.
I think these can be Mapping.	chipx86	March 5, 2026, 3:34 p.m.
Let's make this keyword-only.	chipx86	March 5, 2026, 3:34 p.m.
Missing a Version Added and a Version Changed above. I know it's tests, but it's still nice to have that …	chipx86	March 5, 2026, 3:34 p.m.
Let's make these keyword-only.	chipx86	March 5, 2026, 3:35 p.m.
Missing , optional. Also, same feedback about doc versioning.	chipx86	March 5, 2026, 3:37 p.m.
Can we make these keyword-only?	chipx86	March 5, 2026, 3:37 p.m.
This can be: for dist, dist_info in sorted(DISTS.items()):	chipx86	March 5, 2026, 3:37 p.m.
Seems we shouldn't even bother with the for loop is platforms_set is empty.	chipx86	March 5, 2026, 3:45 p.m.
Blank line here.	chipx86	March 5, 2026, 3:38 p.m.
This should use the standard multi-line format.	chipx86	March 5, 2026, 3:40 p.m.
Can we call this build?	chipx86	March 5, 2026, 3:41 p.m.
We generally do one keyword argument per line.	chipx86	March 5, 2026, 3:41 p.m.

flake8 failed.

JSHint passed.

flake8

tests/run-install-tests.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (83 > 79 characters)

Column: 80
Error code: E501
```

Commits:

	Summary	ID
	Make the (interactive) output of run-install-tests much nicer. Since we have the `rich` library available in the rbinstall context, we might as well use it. This makes it so parallel builds display in a live table. This also adds a `--list` argument which will print a list of the available build targets (in case we want to do something like add this to CI later). Testing Done: Ran install tests and saw much nicer output.	wvypxystvwoszxtpxwuzwulylpptnyzr
	Make the (interactive) output of run-install-tests much nicer. Since we have the `rich` library available in the rbinstall context, we might as well use it. This makes it so parallel builds display in a live table. This also adds a `--list` argument which will print a list of the available build targets (in case we want to do something like add this to CI later). Testing Done: Ran install tests and saw much nicer output.	wvypxystvwoszxtpxwuzwulylpptnyzr

Diff:

Revision 2 (+452 -122)

Show changes

tests/run-install-tests.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Blank line here.
```
tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
I think these can be Mapping.
```
tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Let's make this keyword-only.
```

tests/run-install-tests.py (Diff revision 2)

The issue has been resolved. Show all issues

Missing a Version Added and a Version Changed above.

I know it's tests, but it's still nice to have that information.

tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Let's make these keyword-only.
```
tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Missing , optional.
Also, same feedback about doc versioning.
```
tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Can we make these keyword-only?
```

tests/run-install-tests.py (Diff revision 2)

The issue has been resolved. Show all issues

This can be:

for dist, dist_info in sorted(DISTS.items()):

tests/run-install-tests.py (Diff revision 2)

The issue has been resolved. Show all issues

Seems we shouldn't even bother with the for loop is platforms_set is empty.

david March 5, 2026, 3:45 p.m.

I'll just have the entire thing error out, because that doesn't make sense.

tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Blank line here.
```
tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
This should use the standard multi-line format.
```
tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Can we call this build?
```
tests/run-install-tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
We generally do one keyword argument per line.
```

Commits:

	Summary	ID
	Make the (interactive) output of run-install-tests much nicer. Since we have the `rich` library available in the rbinstall context, we might as well use it. This makes it so parallel builds display in a live table. This also adds a `--list` argument which will print a list of the available build targets (in case we want to do something like add this to CI later). Testing Done: Ran install tests and saw much nicer output.	wvypxystvwoszxtpxwuzwulylpptnyzr
	Make the (interactive) output of run-install-tests much nicer. Since we have the `rich` library available in the rbinstall context, we might as well use it. This makes it so parallel builds display in a live table. This also adds a `--list` argument which will print a list of the available build targets (in case we want to do something like add this to CI later). Testing Done: Ran install tests and saw much nicer output.	wvypxystvwoszxtpxwuzwulylpptnyzr

Diff:

Revision 3 (+532 -134)

Show changes

tests/run-install-tests.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.