Skip to content

Conversation

@friebel
Copy link

@friebel friebel commented Sep 25, 2022

Connection logging after applying these changes:

> rshell --port /dev/cu.usbserial-01EDB820
Connecting to /dev/cu.usbserial-01EDB820 (buffer-size 32) ...
Trying to connect to REPL ... connected
Retrieving sysname ... esp32
Testing if ubinascii.unhexlify exists ... Y
Retrieving root directories ... /testdir/
Setting time ... Sep 25, 2022 09:45:09
Evaluating board_name ... pyboard
Retrieving time epoch ... Jan 01, 2000
Welcome to rshell. Use Control-D (or the exit command) to exit rshell.
  • Buffer size is not logged twice
  • All operation ... result messages follow the same format
  • Root directories list only includes directories, no files

The same information is logged in the called functions, so even
uncommenting the debug prints would not add value.
The buffer size is an implementation detail that should just work out of
the box and does not add value to the average user. Only print it in
debug mode.
Most log messages follow the same format:

  Operation ... result

Modify the two message accordingly that do not follow that format yet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant