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

Remove duplicated test name #8

Merged
merged 2 commits into from Sep 3, 2017

Conversation

Projects
None yet
2 participants
@mathphreak
Contributor

mathphreak commented Sep 1, 2017

Currently, braces_alpha_range1 is used and then immediately reused. I've fixed that, and also renumbered all the other tests.

@@ -201,12 +201,12 @@ new_ec_test(braces_numeric_range7 braces.in 121 "^[ \t\n\r]*$")
new_ec_test(braces_numeric_range8 braces.in 060 "^[ \t\n\r]*$")
# alphabetical brace range
new_ec_test(braces_alpha_range1 braces.in {ardvark..antimater} "^words=a[ \t\n\r]*$")
new_ec_test(braces_alpha_range1 braces.in a "^[ \t\n\r]*$")
new_ec_test(braces_alpha_range1 braces.in {aardvark..antimater} "^words=a[ \t\n\r]*$")

This comment has been minimized.

@xuhdev

xuhdev Sep 2, 2017

Member

Did you intend to change this line?

This comment has been minimized.

@mathphreak

mathphreak Sep 2, 2017

Contributor

Yes, since "aardvark" is spelled correctly on line 206 I assumed it should have been spelled correctly here as well.

@xuhdev

xuhdev approved these changes Sep 3, 2017

@xuhdev xuhdev merged commit 172eb83 into editorconfig:master Sep 3, 2017

@xuhdev

This comment has been minimized.

Member

xuhdev commented Sep 3, 2017

Thank you!

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