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

Restore the correct sorting behavior, add tests to verify it #37

Merged
merged 1 commit into from
Mar 4, 2017

Conversation

matteocng
Copy link
Contributor

I accidentally broke the correct sorting behavior in #36 (sort keys in './lib/Keywords.coffee' > 'sort' before the other keys). 😟

  • Restore the correct sorting behavior.
  • Add two new tests to verify the expected behavior.

- Restore the correct sorting behavior.
- Add two new tests to verify the expected behavior.
@coveralls
Copy link

coveralls commented Mar 4, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 092b504 on matteocng:fix-sort-order into 44930aa on Kikobeats:master.

@Kikobeats Kikobeats merged commit 98859a1 into Kikobeats:master Mar 4, 2017
@matteocng
Copy link
Contributor Author

That was fast, thanks. 😃

@matteocng matteocng deleted the fix-sort-order branch March 4, 2017 21:23
@Kikobeats
Copy link
Owner

💪

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

Successfully merging this pull request may close these issues.

None yet

3 participants