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.

Sign up for GitHub

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

Jump to bottom

refactor(component): use YAML for all component definition files #947

Merged
merged 1 commit into from
Dec 25, 2024
Merged

refactor(component): use YAML for all component definition files #947

merged 1 commit into from
Dec 25, 2024

Conversation

donch1989
Copy link
Member

Because

  • Originally, we used JSON for the component definition files, but its readability is limited since JSON doesn’t support comments or multiline strings.

This commit

  • Converts all component definition files to YAML.

@linear Linear
Copy link

linear bot commented Dec 25, 2024

INS-7126 use YAML for component definition files

@donch1989 donch1989 merged commit 1d7ceb8 into main Dec 25, 2024
12 checks passed
@donch1989 donch1989 deleted the huitang/ins-7126 branch December 25, 2024 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GeorgeWilliamStrong GeorgeWilliamStrong Awaiting requested review from GeorgeWilliamStrong GeorgeWilliamStrong is a code owner

@jvallesm jvallesm Awaiting requested review from jvallesm jvallesm is a code owner

Assignees
No one assigned
Labels
instill component instill vdp
Projects
No open projects
🔮 Instill Core
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@donch1989 @droplet-bot