-
-
Notifications
You must be signed in to change notification settings - Fork 122
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
[data]: Data Preparation for Improved Data Model #738
[data]: Data Preparation for Improved Data Model #738
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think it makes sense to have empty arrays as values.
Where there's no values in an array, let's simply ommit the key.
I appreciate that doing so may have enabled you to more quickly create the data.
Values which can be empty should not be required.
In fact, it may be preferable to not allow empty arrays in the data to avoid possible errors in creating or modifying the data.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job!
What kind of change does this PR introduce?
This PR aims to improve the current data model for the website to enhance maintainability, completeness, and structure. The existing model primarily contains validators' data for the tooling section, while other data is hardcoded into the markdown files. This PR cleans unused and outdated fields, redesign the data model to consolidate all relevant data into a single file, and add maintainer details for future community events.
Issue Number:
Screenshots/videos:
If relevant, did you update the documentation?
Summary
Does this PR introduce a breaking change?