ex172000 commented on PR #3063: URL: https://github.com/apache/iggy/pull/3063#issuecomment-4435843741
Reopening to address the remaining review feedback. Pushed a new commit on top of the rebased branch covering: - chengxilo: `thenStreamDeletedSuccessfully` now verifies via `GetStream` instead of checking the local pointer that `whenDeleteStream` itself nilled. - chengxilo: anchored all `sc.Step` regexes with `^...$` for consistency. - chengxilo: kept the per-scenario `After` cleanup but documented it as best-effort, with a note that a global cleanup script remains the better long-term direction. - hubcio: made the stream-delete step explicit in the shared feature file (`When I delete the stream with name "..."`) and updated the Go, Rust, Python, Java, C#, and Node step definitions to match. Branch is rebased on current `master`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
