Skip to content

Conversation

@capaj
Copy link
Contributor

@capaj capaj commented Oct 11, 2018

fixes #1109

will try to add a test as well, although I am not 100% sure where to put them. Should I create another test filed named #1109.js like couple others there?

@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) to 97.256% when pulling 92daf71 on capaj:fix-1109 into 1569b76 on Vincit:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) to 97.256% when pulling 92daf71 on capaj:fix-1109 into 1569b76 on Vincit:master.

@coveralls
Copy link

coveralls commented Oct 11, 2018

Coverage Status

Coverage increased (+0.06%) to 97.238% when pulling fa123bf on capaj:fix-1109 into 1569b76 on Vincit:master.

@kibertoad
Copy link
Collaborator

@capaj Considering that there is a growing number of these, maybe a separate folder for issue-related tests would make sense?

@capaj
Copy link
Contributor Author

capaj commented Nov 18, 2018

Will try to put it in a new folder then.

@koskimas koskimas closed this in ff40c8e Jan 6, 2019
flipace pushed a commit to flipace/objection.js that referenced this pull request May 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants