CLI Script
other
Feature Proposal
I.e.
This
|
cwms = { |
|
"module": "cwms", |
|
"package": "cwms-python", |
|
"version": "0.8.0", |
|
"desc": "CWMS REST API Python client", |
|
"link": "https://github.com/HydrologicEngineeringCenter/cwms-python", |
|
} |
just gets used on everything via
Thoughts on me making an issue to remove the global and make those explicit?
i.e.
@requries(reqs.cwms.set("version", "1.0.3") This would not work
This will make it so a sub command like users can be set to 1.0.3 but the people also wanting to just have it work for another command can have an older version if they don't need to update!
REDACTED relevant log output
Related Context or Links
There's a global dep in the pytoml here
|
cwms-python = { version = ">=0.8.0", optional = true} |
That could also be at a lower minimum required for all of cwms-cli. Although cwms-cli itself does not need cwms-python until you drill down.
CLI Script
other
Feature Proposal
I.e.
This
cwms-cli/cwmscli/requirements.py
Lines 5 to 11 in 1bedb0f
just gets used on everything via
cwms-cli/cwmscli/commands/commands_cwms.py
Line 28 in 1bedb0f
Thoughts on me making an issue to remove the global and make those explicit?
i.e.
This would not work@requries(reqs.cwms.set("version", "1.0.3")This will make it so a sub command like users can be set to
1.0.3but the people also wanting to just have it work for another command can have an older version if they don't need to update!REDACTED relevant log output
Related Context or Links
There's a global dep in the pytoml here
cwms-cli/pyproject.toml
Line 21 in 1bedb0f
That could also be at a lower minimum required for all of cwms-cli. Although cwms-cli itself does not need cwms-python until you drill down.