Skip to content
This repository was archived by the owner on Jul 11, 2026. It is now read-only.

fix: correct falied to failed typo in monitor.go (4 instances)#25

Open
Jah-yee wants to merge 1 commit into
philsong:masterfrom
Jah-yee:fix/falied-typo
Open

fix: correct falied to failed typo in monitor.go (4 instances)#25
Jah-yee wants to merge 1 commit into
philsong:masterfrom
Jah-yee:fix/falied-typo

Conversation

@Jah-yee

@Jah-yee Jah-yee commented Jul 9, 2026

Copy link
Copy Markdown

Correct falied to failed typo in 4 fmt.Printf statements (src/monitor/monitor.go lines 163, 169, 189, 195). These are user-facing print messages shown when cancel operations fail.

@Jah-yee

Jah-yee commented Jul 9, 2026

Copy link
Copy Markdown
Author

Hi @philsong! This PR fixes 4 instances of falied→failed typo in printf statements — a clean fix in go-signal.go. The PR is mergeable and ready. If it looks good, please consider merging! 👍

@Jah-yee

Jah-yee commented Jul 10, 2026

Copy link
Copy Markdown
Author

Hi @philsong! Gentle reminder — this PR fixes falied→failed in 4 printf statements, clean and mergeable. Ready when you are! 👍

@Jah-yee

Jah-yee commented Jul 10, 2026

Copy link
Copy Markdown
Author

Hi @philsong, just wanted to follow up on this PR. It's been open for a while and I'm happy to make any adjustments if needed. Thanks for considering it!

@Jah-yee

Jah-yee commented Jul 10, 2026

Copy link
Copy Markdown
Author

Hi! This PR is ready to merge. No action needed if it looks good. Thanks! 🙏

@Jah-yee

Jah-yee commented Jul 11, 2026

Copy link
Copy Markdown
Author

Hi @philsong! Bumping this — 4x falied→failed typo fix, CLEAN and mergeable ✅. Would be great to get this merged! Let me know if any changes needed 🙏

@Jah-yee

Jah-yee commented Jul 11, 2026

Copy link
Copy Markdown
Author

Hi! Bumping this fix — let me know if any questions! 👍

@Jah-yee

Jah-yee commented Jul 11, 2026

Copy link
Copy Markdown
Author

Hi! This PR is ready for merge — just a gentle follow-up to see if it needs any adjustments. Happy to help with anything! 🙏

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant