5128755577
82%

Ran 31 May 2023 04:10AM UTC

Jobs 1

Files 15

Run time 1s

Badge

Embed ▾

pending completion

Build # 5128755577

Build Type

push

github

Committed by

bokwoon95

Commit Message

remove SetTimestamp

Users can use Set(NewTimestamp(t)) instead, I may add SetTimestamp back
in the future depending on usage patterns (together with
TimestampParam(name, value), col.SetTimestamp(field, value),
row.Timestamp(format, values...), row.TimestampField(field)). The idea
is that all those values implicitly wrap a time.Time with NewTimestamp()
instead of requiring the user to pass in a Timestamp themselves because
nobody is likely going to have sq.Timestamp in their models. The only
problem is that it then shares the same type signatures as
field.SetTime/TimeParam/col.SetTimestamp/row.Time/row.TimeField so it
could be more confusing than necessary. That's why taking it out first.

Run Details

5387 of 6243 relevant lines covered (86.29%)

49.62 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	5128755577.1	31 May 2023 04:10AM UTC	15	86.29	GitHub Action Run

bokwoon95 / sq / 5128755577
82%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 5128755577

bokwoon95 / sq / 5128755577 82%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 5128755577

bokwoon95 / sq / 5128755577
82%

README BADGES
x