Skip to content

Improve PR approval script logging - #219

Merged
tautschnig merged 2 commits into
model-checking:mainfrom
carolynzech:pr-approvals-debugging
Dec 10, 2024
Merged

Improve PR approval script logging#219
tautschnig merged 2 commits into
model-checking:mainfrom
carolynzech:pr-approvals-debugging

Conversation

@carolynzech

Copy link
Copy Markdown

Improve the logging in our PR approvals workflow. This is a good thing to do generally, but we're specifically trying to debug why the script is failing in #107.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

@carolynzech
carolynzech requested a review from a team as a code ownerDecember 10, 2024 14:53

@feliperodrifeliperodri left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you so much!! 😄

@carolynzech
carolynzech enabled auto-merge (squash) December 10, 2024 15:12
auto-merge was automatically disabled December 10, 2024 17:33

Pull request was closed

@carolynzech
carolynzech deleted the pr-approvals-debugging branch December 10, 2024 17:33
@carolynzech
carolynzech restored the pr-approvals-debugging branch December 10, 2024 17:57
@carolynzech
carolynzech enabled auto-merge (squash) December 10, 2024 17:58

@carolynzechcarolynzech left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

trigger merge

@tautschnig
tautschnig added this pull request to the merge queueDec 10, 2024
Merged via the queue into model-checking:main with commit 7031b93Dec 10, 2024
Sign up for freeto 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

@carolynzech@tautschnig@feliperodri