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
fix some typing mistakes #1023
fix some typing mistakes #1023
Conversation
@yoavf can you check this please |
That's a strange failure on TravisCI, I've restarted the builds to see if it was just temporary, if not I'll take a look in to it some more. |
@@ -697,11 +697,11 @@ | |||
extractor = ts.getParserById( ts.getData( header, configHeaders, 'extractor' ) ); | |||
parser = ts.getParserById( ts.getData( header, configHeaders, 'sorter' ) ); | |||
noParser = ts.getData( header, configHeaders, 'parser' ) === 'false'; | |||
// empty cells behaviour - keeping emptyToBottom for backwards compatibility | |||
// empty cells behavior - keeping emptyToBottom for backwards compatibility |
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.
Not going to block it because of these, but behaviour is a correct spelling... in Canada ;)
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.
Third-party libraries shouldn't be changed at all and reported upstream instead.
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.
Agreed, it didn't even click that it was in a library. There is a new version of the library so I'll create an issue/PR to update it.
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.
+1
fix some typing mistakes