Skip to content

Removing matProps dataSchema#2499

Open
john-science wants to merge 1 commit intomainfrom
rm_data_schema
Open

Removing matProps dataSchema#2499
john-science wants to merge 1 commit intomainfrom
rm_data_schema

Conversation

@john-science
Copy link
Copy Markdown
Member

What is the change? Why is it being made?

The matProps/dataSchema package is not used by matProps, nor is it imported by anyone downstream.

But I have already found that it is mildly out-of-date, and kind of hard to keep track when and where to update it. It's NOT a bad package, but will cause a lot of hard-to-automate work over time. And, in the end, I think what people are really going to want is matProps documentation. It will be more helpful to people to read the docs to see what a matProps YAML file can look like; our users don't want to wade into the dataSchema format JSON file to try to figure it out themselves.

SCR Information

Change Type: trivial

One-Sentence Rationale: We should only include code that our users want.

One-line Impact on Requirements: NA


Checklist

@john-science john-science requested a review from opotowsky April 10, 2026 15:52
@john-science john-science added the cleanup Code/comment cleanup: Low Priority label Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Code/comment cleanup: Low Priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant