Skip to content

Conversation

@d-ragul
Copy link
Contributor

@d-ragul d-ragul commented Feb 7, 2026

This PR fixes a minor documentation typo by replacing British English spelling
with American English spelling for consistency. No functional behavior is changed.

@elliotb-lowrisc
Copy link
Contributor

elliotb-lowrisc commented Feb 9, 2026

Thanks @d-ragul, I'm a big fan of spelling corrections. Could you please add a sign-off line to the commit as per the contributing guidelines in CONTRIBUTING.md, then we can think about merging it

@d-ragul d-ragul force-pushed the doc-fix-behavior-spelling branch from bed9e29 to 1c02b64 Compare February 9, 2026 15:26
@d-ragul
Copy link
Contributor Author

d-ragul commented Feb 9, 2026

Thanks for the review. I’ve added the sign-off as requested.

@elliotb-lowrisc elliotb-lowrisc added the CI:Rerun Rerun failed CI jobs label Feb 9, 2026
@github-actions github-actions bot removed the CI:Rerun Rerun failed CI jobs label Feb 9, 2026
@elliotb-lowrisc elliotb-lowrisc added the CI:Rerun Rerun failed CI jobs label Feb 10, 2026
@github-actions github-actions bot removed the CI:Rerun Rerun failed CI jobs label Feb 10, 2026
Copy link
Contributor

@elliotb-lowrisc elliotb-lowrisc left a comment

Choose a reason for hiding this comment

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

Looks good, thanks

Copy link
Contributor

@rswarbrick rswarbrick left a comment

Choose a reason for hiding this comment

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

Thanks!

@rswarbrick
Copy link
Contributor

The failing CI jobs all seem to be infrastructure problems and clearly can't be caused by this change(!).

@rswarbrick rswarbrick added this pull request to the merge queue Feb 10, 2026
Merged via the queue into lowRISC:master with commit 97b690f Feb 10, 2026
69 of 82 checks passed
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.

3 participants