Skip to content

Fix formatting - #701

Merged
Quinn-With-Two-Ns merged 1 commit into
temporalio:mainfrom
Quinn-With-Two-Ns:run-formatting
Nov 5, 2024
Merged

Fix formatting#701
Quinn-With-Two-Ns merged 1 commit into
temporalio:mainfrom
Quinn-With-Two-Ns:run-formatting

Conversation

@Quinn-With-Two-Ns

@Quinn-With-Two-Ns Quinn-With-Two-Ns commented Nov 4, 2024

Copy link
Copy Markdown
Contributor

@cretz

cretz commented Nov 5, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Hrmm, wonder how it passed in CI branch but failed in main

@Quinn-With-Two-Ns
Quinn-With-Two-Ns merged commit dcee476 into temporalio:main Nov 5, 2024
@Quinn-With-Two-Ns

Quinn-With-Two-Ns commented Nov 5, 2024

Copy link
Copy Markdown
Contributor Author

Yeah I think the issue was I updated the formatter version with another PR in flight

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tsurdilo tsurdilo Awaiting requested review from tsurdilo tsurdilo is a code owner
@antmendoza antmendoza Awaiting requested review from antmendoza antmendoza is a code owner
1 more reviewer
@cretz cretz cretz approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

See More

Successfully merging this pull request may close these issues.

2 participants

@Quinn-With-Two-Ns @cretz