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

Fix wrapping of long messages #102

Open
macpijan opened this issue Jan 30, 2025 · 0 comments
Open

Fix wrapping of long messages #102

macpijan opened this issue Jan 30, 2025 · 0 comments
Assignees
Labels
good first issue Good for newcomers
Milestone

Comments

@macpijan
Copy link
Contributor

macpijan commented Jan 30, 2025

the black autoformatter breaks lines in a way that does not look nice in the terminal:

osfv_cli rte --rte_ip 192.168.10.168 flash probe
DUT model retrieved from snipeit: FW4C
Using rte command is invasive action, checking first if the device is not used...
Asset 42 is already checked out by you
Probing flash...
Executing command: flashrom -p linux_spi:dev=/dev/spidev1.0,spispeed=16000 
flashrom v1.3.0 on Linux 5.4.69 (armv7l)
flashrom is free software, get the source code at https://flashrom.org

Using clock_gettime for delay loops (clk_id: 1, resolution: 1ns).
No EEPROM/flash device found.
Note: flashrom can never write if the flash chip isn't found automatically.
Since the asset 42 has been checkout manually                     by you prior running this script, it will NOT be checked                     in automatically. Please return the device when work is                     finished.

We should change it in a way to:

  • fit the requirements of PEP8
  • look pleasant in the terminal (no extra whitespaces)
@macpijan macpijan added the good first issue Good for newcomers label Feb 14, 2025
@macpijan macpijan added this to the v0.6.0 milestone Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants