Skip to content
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.

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

Create term ID reservation script #2

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

jamesaoverton
Copy link
Member

Fixes #1

@jamesaoverton
Copy link
Member Author

jamesaoverton commented Feb 24, 2020

Very good! I'd like some changes mostly to make this more generic and not OBI-specific:

  1. I think I want a configuration YAML file. It should not contain secrets. Ideally we could reuse ODK YAML config files like these by reusing their keys: id, github_org, repo. I guess we will needs an optional --config FILE argument, but the default should be to look in the same directory as this script for gizmos.yml. GITHUB_REPO and GITHUB_OWNER must be in there or there's an error. GITHUB_BRANCH, PUBLISHED_FILE, and RESERVED_FILE might be in there, but with the current values as defaults.
  2. The term ID pattern must be configurable, minimally with the IDSPACE (e.g. OBI) and the number of digits (e.g. 7), but there might be a more clever way to do it. The IDSPACE must be in the config file. The digits can be specified in the config or default to 7.
  3. Please change --commit_message to --message
  4. Please change --terms to --labels
  5. Please add short options for everything: -m -l -i -c
  6. Can we optionally read from STDIN?

…ames, idspace and id digits to config file
@lmcmicu
Copy link
Collaborator

lmcmicu commented Feb 26, 2020

@jamesaoverton Please review

@jamesaoverton jamesaoverton marked this pull request as draft May 11, 2021 16:22
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.

Script to reserve ontology term IDs
2 participants