Skip to content

Allow test runner script to increase the default width, with a minimum #269

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

Closed
ianfixes opened this issue Jan 8, 2021 · 0 comments · Fixed by #334
Closed

Allow test runner script to increase the default width, with a minimum #269

ianfixes opened this issue Jan 8, 2021 · 0 comments · Fixed by #334
Labels
ci scripts The test runner scripts enhancement New feature or request

Comments

@ianfixes
Copy link
Collaborator

ianfixes commented Jan 8, 2021

Feature Request

The arduino_ci.rb test runner script hard-codes WIDTH = 80 as a target text formatting for all its checkmarks, x's, and informational outputs.

80 should definitely be the minimum, but we should explore options for letting the user decide on a larger width as per their preferences. It's unclear whether we should attempt something like tput cols (which won't work on Windows) or leave that to the user to simply collect that value on their own and apply it.

An environment variable is probably the easiest and most appropriate way to accomplish this.

@ianfixes ianfixes added enhancement New feature or request ci scripts The test runner scripts labels Jan 8, 2021
@ianfixes ianfixes added this to the 2021 Beta 1 Features milestone Apr 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci scripts The test runner scripts enhancement New feature or request
Projects
None yet
1 participant