Skip to content

Conversation

@ludamad
Copy link
Collaborator

@ludamad ludamad commented Oct 23, 2025

automatically post/update a comment on pull requests listing all flaked tests

@ludamad ludamad marked this pull request as ready for review October 24, 2025 01:25
@ludamad ludamad requested a review from charlielye as a code owner October 24, 2025 01:25
@ludamad ludamad enabled auto-merge October 24, 2025 20:13
automatically post/update a comment on pull requests listing all flaked tests
@AztecBot AztecBot force-pushed the ad/feat/ci3-flake-detection-comments branch from 80f2d36 to 4c9c4a3 Compare October 24, 2025 20:15
Copy link
Contributor

@spalladino spalladino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC there was a gh action for managing sticky comments? Still, if we want to handle everything from sh, this makes more sense.

@ludamad ludamad added this pull request to the merge queue Oct 24, 2025
@ludamad
Copy link
Collaborator Author

ludamad commented Oct 24, 2025

I really think this is the right way of doing things. I was able to test all the pieces working, our mainframe env is pretty much identical to our CI env. IOU: something better than shell. For now, I'm experimenting with new stuff in shell-y Python. (Basically anything we use will be calling out to other programs a lot, though, so I'm not giving huge promises here about being 100% idiomatic in whatever we may decide to start using) Practically speaking, with our current setup it's not worth communicating between the runner and github actions.

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 24, 2025
@ludamad ludamad merged commit 9fab6cc into next Oct 24, 2025
14 checks passed
@ludamad ludamad deleted the ad/feat/ci3-flake-detection-comments branch October 24, 2025 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants