Skip to content

Conversation

@hermanschaaf
Copy link
Member

We should check the first record higher up, because after the migration it will have an additional column and is therefore bound to differ from the original record.

@hermanschaaf hermanschaaf requested a review from yevgenypats as a code owner July 4, 2023 15:34
@github-actions github-actions bot added the fix label Jul 4, 2023
@kodiakhq kodiakhq bot merged commit a9bd88f into main Jul 4, 2023
@kodiakhq kodiakhq bot deleted the fix-tests-some-more branch July 4, 2023 15:55
kodiakhq bot pushed a commit that referenced this pull request Jul 4, 2023
🤖 I have created a release *beep* *boop*
---


## [4.7.1-rc1](v4.7.0-rc1...v4.7.1-rc1) (2023-07-04)


### Bug Fixes

* Add AddCqIDs helper function ([#1065](#1065)) ([911762d](911762d))
* Check record data in tests ([#1062](#1062)) ([f13e4cc](f13e4cc))
* **configtype:** Add `Equal()` method to `Duration` ([#1059](#1059)) ([57c7bc2](57c7bc2))
* Conversion and test fixes ([#1064](#1064)) ([36b65cb](36b65cb))
* Fix test assertions for records ([#1066](#1066)) ([a9bd88f](a9bd88f))
* **testdata:** Add missing column types ([#1061](#1061)) ([f5d01c9](f5d01c9))

---
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.

3 participants