influx-query-builder icon indicating copy to clipboard operation
influx-query-builder copied to clipboard

Published 20 hours ago • verified publisher icon benjamin658

fix support of unquoted value on time criteria

Open fifthaxe opened this issue 5 years ago • 2 comments

influxql permits to set an unquoted string for a timestamp : time >= 12ms

this PR adds support to this case

fifthaxe avatar Dec 20 '19 23:12 fifthaxe

Pull Request Test Coverage Report for Build 71

  • 2 of 6 (33.33%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.2%) to 98.758%
Changes Missing Coverage Covered Lines Changed/Added Lines %
query_builder.go 2 6 33.33%
<!-- Total: 2 6
Totals Coverage Status
Change from base Build 65: -1.2%
Covered Lines: 318
Relevant Lines: 322
💛 - Coveralls

coveralls avatar Dec 20 '19 23:12 coveralls

Thanks for your contribution, it would be great to add unit test on it, after the test passed, I will merge the PR, thanks again!

benjamin658 avatar Dec 22 '19 09:12 benjamin658