Skip to content

fw_upgrade: fix 'occured' -> 'occurred' in upgrade log/error messages#264

Closed
SAY-5 wants to merge 2 commits intofacebook:heliumfrom
SAY-5:fix-fw-upgrade-occured-typos
Closed

fw_upgrade: fix 'occured' -> 'occurred' in upgrade log/error messages#264
SAY-5 wants to merge 2 commits intofacebook:heliumfrom
SAY-5:fix-fw-upgrade-occured-typos

Conversation

@SAY-5
Copy link
Copy Markdown

@SAY-5 SAY-5 commented Apr 16, 2026

Five log/error messages across two firmware upgrade files read occured:

File Lines Context
tools/fw_upgrade/entity_upgrader.py 251, 278, 384, 435 Exception and error return code messages visible in upgrade logs
tests2/common/base_fw_upgrade_test.py 532 Test log message

Fixed to occurred. String-literal-only change — visible in firmware upgrade output logs.

SAY-5 added 2 commits April 16, 2026 13:11
Four log/error messages in tools/fw_upgrade/entity_upgrader.py and one in tests2/common/base_fw_upgrade_test.py read 'occured'. User-visible in upgrade logs. Fixed to 'occurred'. String-literal-only change.

Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
@meta-cla meta-cla Bot added the CLA Signed label Apr 16, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Apr 16, 2026

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this in D101227204. (Because this pull request was imported automatically, there will not be any future comments.)

@SAY-5
Copy link
Copy Markdown
Author

SAY-5 commented Apr 28, 2026

Closing — typo-only PR. Multiple maintainers across the OSS ecosystem have flagged my recent typo-sweep PRs as AI-generated spam (notably hashicorp/nomad#27855, hashicorp/hcl#794, argo-cd reviewers). I should have caught this pattern sooner. Apologies for the noise.

@SAY-5 SAY-5 closed this Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant