Smithsonian / Smithsonian/caas-aspace-scripts
[FEATURE] Add dry-run or test Argument to update_locations.py
@crugas is already working on this.
Since Jan 24, 2025.
- Dominant language
- Python
- Stars
- 1
- Forks
- 0
- Avg merge
- 4d 5h
- Merged PRs (30d)
- 1
Description
Is your feature request related to a problem? Please describe.
Add a dry-run or test argument to update_locations.py, just like what's done here: update_fileuri.py line 28. This will better enable testing of the script to run without actually changing data in test (or more importantly prod).
Describe the solution you'd like
Either mimicking the above with a dedication function to handle and parse script arguments or a user input function.
Describe alternatives you've considered
I think argparse would be the first choice. Another option could be input, but would need a lot more guardrails around user input and verifying that it works.
Additional context
Next time we need to update locations for an SI unit, consider including this as part of the development workflow.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.