Thanks to visit codestin.com
Credit goes to Github.com

Skip to content

Comments

Allow torq config pull to write to non-existent files#9

Merged
jahdiel-alvarez merged 22 commits intomainfrom
pull-create-file
Jan 26, 2026
Merged

Allow torq config pull to write to non-existent files#9
jahdiel-alvarez merged 22 commits intomainfrom
pull-create-file

Conversation

@samueldjohnson
Copy link
Collaborator

@samueldjohnson samueldjohnson commented Jan 12, 2026

  • Currently, 'torq config pull' will only write to existing files. Update the logic to write to a new file if the provided file doesn't exist. As well, add ".txtpb" to the end of the filename if it isn't already there.
  • Update 'torq config pull' to check with the user that overwriting an existing file is ok.
  • Update config.py to make args.file_path a pathlib.Path object.
  • Add and update relevant tests. Update config_command_executor tests to use run_cli.
  • Update run_cli and the run method to be able to pass a mock_device.

Test: ./torq_test
Fixes: 474686851

Currently, 'torq config pull' will only write to existing files. Update
the logic to write to a new file if the provided file doesn't exist. As
well, add ".pbtxt" to the end of the filename if it isn't already there.

Test: ./torq_test
Fixes: 474686851
@jahdiel-alvarez jahdiel-alvarez merged commit 4279f0a into main Jan 26, 2026
4 checks passed
@samueldjohnson samueldjohnson deleted the pull-create-file branch January 26, 2026 23:04
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.

2 participants