Skip to content

Conversation

@marianogappa
Copy link
Contributor

Fix bug discovered in this Slack thread: https://cloudqueryio.slack.com/archives/C0774DTKDJT/p1719328163706899

Which discusses this issue:

https://github.com/cloudquery/cloudquery-issues/issues/1911

Which says:

sync_run_completed event is sent with status "success" and with zero errors

@marianogappa marianogappa changed the title Fix hardcoded status property on sync_run_completed. fix: Fix hardcoded status property on sync_run_completed. Jun 25, 2024
@marianogappa marianogappa marked this pull request as ready for review June 25, 2024 15:53
@marianogappa marianogappa requested review from a team and bbernays and removed request for a team June 25, 2024 15:53
@marianogappa marianogappa added the automerge Automatically merge once required checks pass label Jun 25, 2024
@kodiakhq kodiakhq bot merged commit 024a7ae into main Jun 25, 2024
@kodiakhq kodiakhq bot deleted the mariano/fix-success-hardcode branch June 25, 2024 17:04
kodiakhq bot pushed a commit that referenced this pull request Jun 26, 2024
🤖 I have created a release *beep* *boop*
---


## [5.23.1](cli-v5.23.0...cli-v5.23.1) (2024-06-25)


### Bug Fixes

* **deps:** Update module github.com/hashicorp/go-retryablehttp to v0.7.7 [SECURITY] ([#18369](#18369)) ([1e223bf](1e223bf))
* Fix hardcoded status property on sync_run_completed. ([#18372](#18372)) ([024a7ae](024a7ae))

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

Labels

area/cli automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants