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

Ensure 🔤 in api.yml #13846

Merged
merged 1 commit into from
Feb 9, 2017
Merged

Ensure 🔤 in api.yml #13846

merged 1 commit into from
Feb 9, 2017

Conversation

imtayadeway
Copy link
Contributor

@imtayadeway imtayadeway commented Feb 9, 2017

Puts the api.yml collections in alphabetical order and adds a test to
prevent regressions.

Not a big fan of these kinds of tests but I think at 2276 lines it's hard to maintain this file by eye.

@miq-bot add-label api, refactor
@miq-bot assign @abellotti

Puts the api.yml collections in alphabetical order and adds a test to
prevent regressions.
@miq-bot
Copy link
Member

miq-bot commented Feb 9, 2017

@imtayadeway Cannot apply the following label because they are not recognized: refactor

@abellotti
Copy link
Member

This is good to have for our sanity, so 👍 waiting for 🍏

@miq-bot
Copy link
Member

miq-bot commented Feb 9, 2017

Checked commit imtayadeway@fd5426d with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks good. 🍪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants