Smithsonian / Smithsonian/caas-aspace-scripts

[FEATURE] Add dry-run or test Argument to update_locations.py

Open
#26 0 comments 0 reactions 1 assignee View on GitHub

@crugas is already working on this.

Since Jan 24, 2025.

enhancement
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

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.