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.

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

fix(testing): Comply with given TimePrecision #1089

Merged
merged 1 commit into from
Jul 13, 2023

Conversation

disq
Copy link
Member

@disq disq commented Jul 12, 2023

Turns out TP isn't observed in InsertAll/UpsertAll. Needed for cloudquery/cloudquery#12145

@disq disq requested a review from hermanschaaf July 12, 2023 23:12
@disq disq requested a review from yevgenypats as a code owner July 12, 2023 23:12
@github-actions github-actions bot added the fix label Jul 12, 2023
@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (6238bfb) 47.27% compared to head (e87946c) 47.27%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1089   +/-   ##
=======================================
  Coverage   47.27%   47.27%           
=======================================
  Files          82       82           
  Lines        7713     7713           
=======================================
  Hits         3646     3646           
  Misses       3733     3733           
  Partials      334      334           
Impacted Files Coverage Δ
plugin/testing_upsert.go 0.00% <0.00%> (ø)
plugin/testing_write_insert.go 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions
Copy link

⏱️ Benchmark results

Comparing with 6238bfb

  • Glob-8 ns/op: 98.47 ⬇️ 1.66% decrease vs. 6238bfb

@disq disq added the automerge label Jul 13, 2023
@disq disq merged commit d16ed0f into cloudquery:main Jul 13, 2023
7 checks passed
@disq disq deleted the fix/testing-timeprec branch July 13, 2023 12:37
kodiakhq bot pushed a commit that referenced this pull request Jul 14, 2023
🤖 I have created a release *beep* *boop*
---


## [4.1.0](v4.0.0...v4.1.0) (2023-07-14)


### Features

* Add `plugin.ValidateNoEmptyColumns` ([#1085](#1085)) ([32e1215](32e1215))


### Bug Fixes

* Add random suffix to test table names ([#1086](#1086)) ([ad16b20](ad16b20))
* **deps:** Update module github.com/cloudquery/plugin-pb-go to v1.7.0 ([#1091](#1091)) ([fb124a2](fb124a2))
* **testing:** Comply with given TimePrecision ([#1089](#1089)) ([d16ed0f](d16ed0f))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants