Skip to content
This repository was archived by the owner on Jul 16, 2020. It is now read-only.

Conversation

jsayol
Copy link
Contributor

@jsayol jsayol commented Nov 2, 2016

The message suggests to add an ._indexOn rule but that should actually say .indexOn. Additionally, make it clear that no action needs to be taken if the rules have already been added.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 9422691 on jsayol:patch-1 into 6e829ee on davideast:master.

@davideast
Copy link
Owner

Wow! Super fast @jsayol! Can you also change the comment for the function as well?

https://github.com/jsayol/Querybase/blob/942269154c9634645a88126c32d2efd69cfc1f92/src/querybase.ts#L623

@jsayol
Copy link
Contributor Author

jsayol commented Nov 2, 2016

Woops, missed it.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 4efb723 on jsayol:patch-1 into 6e829ee on davideast:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 997ee1b on jsayol:patch-1 into 6e829ee on davideast:master.

@jsayol
Copy link
Contributor Author

jsayol commented Nov 2, 2016

Dammit, I'm editing the file directly from inside GitHub and the built-in editor insists on adding a newline character at the end of the file. I guess it's staying.

Fixed it, I don't like half-assing things.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 0f9334f on jsayol:patch-1 into 6e829ee on davideast:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 0f9334f on jsayol:patch-1 into 6e829ee on davideast:master.

@jsayol
Copy link
Contributor Author

jsayol commented Nov 2, 2016

I removed the extra newline and squashed it all in one single commit. Sorry for all the noise!

The message suggest to add an `._indexOn` rule but that should actually say `.indexOn`. Additionally, make it clear that no action needs to be taken if the rules have already been added.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d47b0b2 on jsayol:patch-1 into 6e829ee on davideast:master.

@davideast davideast merged commit 9bc48e7 into davideast:master Nov 2, 2016
@davideast
Copy link
Owner

Thank you so much @jsayol!

@jsayol
Copy link
Contributor Author

jsayol commented Nov 2, 2016

Just realized I missed a typo on my own commit message. Oh well.

davideast pushed a commit that referenced this pull request Nov 28, 2016
The message suggest to add an `._indexOn` rule but that should actually say `.indexOn`. Additionally, make it clear that no action needs to be taken if the rules have already been added.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants