Skip to content

fix: Data race when reporting progress#21117

Merged
kodiakhq[bot] merged 1 commit intomainfrom
fix/data_race_debouncer
Jul 24, 2025
Merged

fix: Data race when reporting progress#21117
kodiakhq[bot] merged 1 commit intomainfrom
fix/data_race_debouncer

Conversation

@erezrokah
Copy link
Copy Markdown
Member

@erezrokah erezrokah commented Jul 24, 2025

Summary

Should fix the data race in #21116. Instead of using the Done() channel we can call the debounced function one last time.

The data race is a known issue in the package, see vnteamopen/godebouncer#28

@erezrokah erezrokah requested a review from a team as a code owner July 24, 2025 15:30
@erezrokah erezrokah requested a review from mnorbury July 24, 2025 15:30
@erezrokah erezrokah requested a review from disq July 24, 2025 15:31
@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Jul 24, 2025
@kodiakhq kodiakhq bot merged commit 640b94e into main Jul 24, 2025
22 checks passed
@kodiakhq kodiakhq bot deleted the fix/data_race_debouncer branch July 24, 2025 15:55
kodiakhq bot pushed a commit that referenced this pull request Jul 24, 2025
🤖 I have created a release *beep* *boop*
---


## [6.25.1](cli-v6.25.0...cli-v6.25.1) (2025-07-24)


### Bug Fixes

* Data race when reporting progress ([#21117](#21117)) ([640b94e](640b94e))

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

3 participants