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-48803: Add workflow for checking consistency of band columns #313

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

Conversation

JeremyMcCormick
Copy link
Collaborator

@JeremyMcCormick JeremyMcCormick commented Mar 4, 2025

This PR adds a workflow for checking the consistency of band column definitions in the following schemas:

  • dp02_dc2
  • dp03*
  • imsim
  • hsc
  • apdb

Several inconsistencies in a few of these schemas were fixed, so the check currently passes.

Introduction of any inconsistencies between band column definitions will cause the workflow and PR status check to fail.


The tools for checking the band columns are introduced by this PR in sdm_tools, which needs to be merged before this one.

This PR will also need to be updated to use the main branch of sdm_tools in requirements.txt instead of a ticket branch.

Checklist

When making changes to YAML files in the schemas directory:

  • If applicable, incremented the schema version number, following the guidelines in the contribution guide
  • Referred to the documentation on specific schemas for additional versioning information, change constraints, or tasks that may need to be performed, based on which schema is being updated

@JeremyMcCormick JeremyMcCormick force-pushed the tickets/DM-48803 branch 3 times, most recently from b8bfdc0 to 09187f0 Compare March 5, 2025 04:04
@JeremyMcCormick JeremyMcCormick force-pushed the tickets/DM-48803 branch 7 times, most recently from 56b3fe9 to 45d2aae Compare March 5, 2025 20:59
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.

1 participant