Skip to content

Paths API Commands in Tethys CLI #1205

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.

Sign up for GitHub

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

Jump to bottom
Merged
swainn merged 19 commits into from
Nov 18, 2025
Merged

Paths API Commands in Tethys CLI #1205

swainn merged 19 commits into from
Nov 18, 2025

Conversation

@jakeymac
Copy link
Contributor

@jakeymac jakeymac commented Oct 21, 2025 •
edited
Loading

Description

This pull request adds new functionality to Tethys CLI, allowing users to:

  • Retrieve file paths in the Tethys Paths API
  • Add files easily to file paths in the Tethys Paths API

It also provides small updates to the Paths API workspace functions

Changes Made to Code

  • Added new commands to Tethys CLI
  • Small updates to Paths API in workspace functions to accept TethysApp objects/models

Related PRs, Issues, and Discussions

Quality Checks

  • At least one new test has been written for new code
  • New code has 100% test coverage
  • Code has been formatted with Black
  • Code has been linted with flake8
  • Docstrings for new methods have been added
  • The documentation has been updated appropriately

@coveralls
Copy link

coveralls commented Nov 5, 2025 •
edited
Loading

Coverage Status

coverage: 100.0%. remained the same
when pulling 9c3af09 on paths-commands
into 4fafd04 on main.

swainn
swainn requested changes Nov 17, 2025
View reviewed changes
@jakeymac
* Added new Paths commands to Tethys CLI to find and add to file path…
7eb35b4 
…s in the Paths API

* Updated functionality of Paths API to accept TethysApp in workspace functions
@jakeymac
Tests fixed
1703596
@jakeymac
Updated error messages and tests
a089a7e
@jakeymac
More test fixes
13c686b
@jakeymac
More test fixes
3f233d7
@jakeymac
More test fixes/adjustments in tests
feb7205
@jakeymac
Fixed tests
8e2a8ef
@jakeymac
* Updated paths command to account for quotas in workspaces/media
1c1553d 
* Updated storage unit conversion helper function and added new can_add_file_to_path helper function in tethys quotas utilities
@jakeymac
* Added test for new path commands
6e247f6 
* Updated path commands with minor adjustments in CLI outputs
@jakeymac
Formatting fix
8d16b50
@jakeymac
Testing fix for failing tests
e28b1d4
@jakeymac
Fixes in tests failing in CI
134e5d1
@jakeymac
* Added tests for new helper functions
8123418 
* Added _convert_to_bytes helper function
@jakeymac
Formatting fixes
cb57e77
@jakeymac
Added documentation for new Paths commands
bd71128
@jakeymac
* Updated output of Paths command for user paths (workspace, media)
5d1af0d 
* Updated Paths commands tests
@jakeymac
Updated documentation with formatted examples of Tethys Paths command…
9960ed2 
… usage
@jakeymac
Updated doc strings in workspace functions
2f26a05
@swainn swainn merged commit f5759ae into main Nov 18, 2025
29 checks passed
@swainn swainn deleted the paths-commands branch November 18, 2025 15:55
@jakeymac jakeymac linked an issue Dec 5, 2025 that may be closed by this pull request
[FEATURE] Workspaces CLI Command #1196
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@swainn swainn swainn approved these changes

Assignees

@jakeymac jakeymac

Labels

major feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FEATURE] Workspaces CLI Command

4 participants

@jakeymac @coveralls @swainn