Skip to content

Conversation

@candiduslynx
Copy link
Contributor

@candiduslynx candiduslynx commented Aug 18, 2023

Part of cloudquery/cloudquery#13153

This changes the semantics of testdata.Generate from generate opts.MaxRows records to generate a single record with opts.MaxRows rows

@candiduslynx candiduslynx marked this pull request as ready for review August 18, 2023 14:00
@github-actions
Copy link

github-actions bot commented Aug 18, 2023

⏱️ Benchmark results

  • Glob-8 ns/op: 99.01

@github-actions github-actions bot added fix and removed fix labels Aug 18, 2023
@codecov
Copy link

codecov bot commented Aug 18, 2023

Codecov Report

Patch coverage: 8.86% and project coverage change: -0.16% ⚠️

Comparison is base (6180bbc) 48.87% compared to head (1ce3ff6) 48.71%.

❗ Current head 1ce3ff6 differs from pull request most recent head 539ff75. Consider uploading reports for the commit 539ff75 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1153      +/-   ##
==========================================
- Coverage   48.87%   48.71%   -0.16%     
==========================================
  Files          87       87              
  Lines        8037     8071      +34     
==========================================
+ Hits         3928     3932       +4     
- Misses       3749     3781      +32     
+ Partials      360      358       -2     
Files Changed Coverage Δ
plugin/diff.go 0.00% <0.00%> (ø)
plugin/testing_upsert.go 0.00% <0.00%> (ø)
plugin/testing_write_delete.go 0.00% <0.00%> (ø)
plugin/testing_write_insert.go 0.00% <0.00%> (ø)
plugin/testing_write_migrate.go 0.00% <0.00%> (ø)
schema/testdata.go 39.54% <36.84%> (-0.69%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kodiakhq kodiakhq bot merged commit 86e717a into main Aug 21, 2023
@kodiakhq kodiakhq bot deleted the chore/multi-row-test-records branch August 21, 2023 10:22
kodiakhq bot pushed a commit that referenced this pull request Aug 21, 2023
🤖 I have created a release *beep* *boop*
---


## [4.5.2](v4.5.1...v4.5.2) (2023-08-21)


### Bug Fixes

* Change `testdata.Generate` signature ([#1153](#1153)) ([86e717a](86e717a))
* **deps:** Update github.com/cloudquery/arrow/go/v13 digest to 5b83d4f ([#1154](#1154)) ([8558dd1](8558dd1))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.5.1 ([#1150](#1150)) ([b3f41b1](b3f41b1))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
@candiduslynx candiduslynx self-assigned this Aug 21, 2023
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.

3 participants