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

DM-41290: Add initial version of sdm_tools #1

Open
wants to merge 30 commits into
base: main
Choose a base branch
from

Conversation

JeremyMcCormick
Copy link
Collaborator

This PR implements an initial version of the sdm_tools repository, which is intended to contain scripts and utilities for working with sdm_schemas.

The building and packaging of the datalink metadata, formerly using scripts in the datalink directory of sdm_schemas, is the first use case that is implemented.

A sdm-tools command line interface is provided where various commands can be added, e.g., the datalink command can be executed using sdm-tools build-datalink-metadata.

@JeremyMcCormick JeremyMcCormick requested a review from gpdf February 7, 2025 22:36
@JeremyMcCormick JeremyMcCormick marked this pull request as draft February 7, 2025 22:55
@JeremyMcCormick JeremyMcCormick force-pushed the tickets/DM-41290 branch 4 times, most recently from cf3fdf2 to a164e07 Compare February 13, 2025 17:53
@JeremyMcCormick JeremyMcCormick marked this pull request as ready for review February 13, 2025 17:53
@JeremyMcCormick JeremyMcCormick changed the title DM-41290: Add initial version of sdm-tools DM-41290: Add initial version Feb 13, 2025
@JeremyMcCormick JeremyMcCormick force-pushed the tickets/DM-41290 branch 4 times, most recently from d271c7e to 328ade1 Compare February 13, 2025 18:09
@JeremyMcCormick JeremyMcCormick requested review from timj and removed request for gpdf February 19, 2025 19:05
@JeremyMcCormick JeremyMcCormick changed the title DM-41290: Add initial version DM-41290: Add initial version of sdm_tools Feb 19, 2025
Copy link
Member

@timj timj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not approving just yet because I have lots of comments and have some questions as to what is public API and what isn't.

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.

2 participants