Skip to content

fix: Set all fields in DeleteRecord message#1316

Merged
kodiakhq[bot] merged 1 commit intocloudquery:mainfrom
bbernays:fix-write
Oct 18, 2023
Merged

fix: Set all fields in DeleteRecord message#1316
kodiakhq[bot] merged 1 commit intocloudquery:mainfrom
bbernays:fix-write

Conversation

@bbernays
Copy link
Contributor

Summary


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests 🧪
  • Ensure the status checks below are successful ✅

@github-actions
Copy link

⏱️ Benchmark results

Comparing with e9c751c

  • Glob-8 ns/op: 90.81 ⬆️ 0.06% increase vs. e9c751c

@kodiakhq kodiakhq bot merged commit ad9d109 into cloudquery:main Oct 18, 2023
@bbernays bbernays deleted the fix-write branch October 18, 2023 20:57
kodiakhq bot pushed a commit that referenced this pull request Oct 18, 2023
🤖 I have created a release *beep* *boop*
---


## [4.15.3](v4.15.2...v4.15.3) (2023-10-18)


### Bug Fixes

* Set all fields in `DeleteRecord` message ([#1316](#1316)) ([ad9d109](ad9d109))

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