-
Notifications
You must be signed in to change notification settings - Fork 22
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
Change for snake_case consistency #151
Conversation
Good catch, @Vidushi-GitHub. Before we merge this, I'd like to make sure that our validation script catches case mismatches in |
OK, our validation script is case sensitive. No problem there. |
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.
This looks fine to me, but we need someone from BurstCube to be aware of the change. Please find and add an appropriate BurstCube team member as a reviewer.
This one is Glowbug, they aren't active. @jracusin What should be done? |
Sorry, of course I meant Glowbug. Is that Boyan? |
Yes, Boyan created this one. |
@shb46, would you please review this? |
@shb46 although you aren't working on this anymore. I am just closing the old PR, could you please approve (if looks fine) this? |
@Vidushi-GitHub: Thanks for fixing the typo. |
Description
Change in mission schema name "Capital" to "Lower case".
It's getting validated for $schema properties.