Skip to content

Add missing apostrophe to WorkflowException message#2776

Merged
tconley1428 merged 3 commits intotemporalio:masterfrom
brempusz:patch-1
Feb 26, 2026
Merged

Add missing apostrophe to WorkflowException message#2776
tconley1428 merged 3 commits intotemporalio:masterfrom
brempusz:patch-1

Conversation

@brempusz
Copy link
Contributor

@brempusz brempusz commented Feb 7, 2026

What was changed

Add missing apostrophe to WorkflowException message

Why?

Because the formatting was wrong

@brempusz brempusz requested a review from a team as a code owner February 7, 2026 21:08
@CLAassistant
Copy link

CLAassistant commented Feb 7, 2026

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@mjameswh
Copy link
Contributor

@brempusz Can you please sign the CLA? We can't examine your PR until that has been cleared.

@brempusz
Copy link
Contributor Author

I did it right after creating the pr but there is something wrong - I see two comments: 1st with information that all commiters signed the CLA and green checkmark and 2nd with yellow checkmark and signing instruction. I tried to sign it again. I tried to "recheck" the status. Nothing works...

@brempusz
Copy link
Contributor Author

After some googling I followed a random recommendation on the net and rebased / pushed the PR again (it may refresh cla status on the commit).
In the checks i see:
image

It is amusing how much time and effort is needed to add just one missing character :)

@tconley1428
Copy link
Contributor

Potentially those github outages had some impact on it. Sorry for the trouble, and thanks for the effort.

@brempusz
Copy link
Contributor Author

@mjameswh, all checks have passed. Could you verify if you can see my CLA signature and merge this PR?

@brempusz
Copy link
Contributor Author

Guys, is there a way to rerun the checks?

@tconley1428 tconley1428 merged commit 9799bd0 into temporalio:master Feb 26, 2026
15 checks passed
@brempusz brempusz deleted the patch-1 branch February 27, 2026 09:55
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.

4 participants