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.

Sign up for GitHub

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

Jump to bottom

correct definition of includes() #22

Closed
hamidyou opened this issue Oct 3, 2018 · 1 comment
Closed

correct definition of includes() #22

hamidyou opened this issue Oct 3, 2018 · 1 comment
Assignees
@dhershman1
Labels
documentation general documentation problem (grammar, incorrect, etc) kyanite related to the kyanite module

Comments

@hamidyou
Copy link

hamidyou commented Oct 3, 2018 •
edited
Loading

The word "at" is repeated and one should be removed.

Also, in the Arguments section, the type of list should be array or string.

screen shot 2018-10-02 at 11 37 07 pm
@dhershman1 dhershman1 added the documentation general documentation problem (grammar, incorrect, etc) label Oct 5, 2018
@dhershman1 dhershman1 self-assigned this Oct 5, 2018
@dhershman1 dhershman1 added the kyanite related to the kyanite module label Oct 5, 2018
@dhershman1
Copy link
Owner

This has been addressed as well as the bad math on the compose function. Fixed with release v0.9.2 of kyanite.

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

@dhershman1 dhershman1

Labels
documentation general documentation problem (grammar, incorrect, etc) kyanite related to the kyanite module
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dhershman1 @hamidyou