Skip to content

fix: remove trailing whitespace in CONTRIBUTING.md#29038

Closed
FtlC-ian wants to merge 1 commit intoopenclaw:mainfrom
FtlC-ian:fix/contributing-trailing-whitespace
Closed

fix: remove trailing whitespace in CONTRIBUTING.md#29038
FtlC-ian wants to merge 1 commit intoopenclaw:mainfrom
FtlC-ian:fix/contributing-trailing-whitespace

Conversation

@FtlC-ian
Copy link
Copy Markdown

Removes 4 trailing spaces on a blank line (line 61) that cause oxfmt --check to fail on every PR — including #15051, which has nothing to do with this file.

One-character fix, zero logic change.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Feb 27, 2026

Greptile Summary

Removes trailing whitespace from line 61 in CONTRIBUTING.md to fix oxfmt --check failures.

  • Removed 4 trailing spaces from blank line after Jonathan Taylor's contributor entry
  • Zero logic change, pure formatting fix
  • Allows formatter checks to pass for all PRs

No issues found.

Confidence Score: 5/5

  • This PR is completely safe to merge with zero risk
  • Simple whitespace-only change that fixes formatter checks. No logic changes, no functional impact, and well-documented in PR description.
  • No files require special attention

Last reviewed commit: 4b24cd0

Copy link
Copy Markdown
Contributor

@jalehman jalehman left a comment

Choose a reason for hiding this comment

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

This fix is incomplete — stripping the trailing spaces from the blank line (line 61) isn't sufficient. oxfmt --check still fails after this change because:

  1. oxfmt wants the blank line deleted entirely, not just stripped
  2. There's a second issue on line 63: the underscore in @jlehman_ needs escaping → @jlehman\_

PR #29040 addresses both and passes oxfmt --check. Recommend closing this in favor of that one.

@FtlC-ian
Copy link
Copy Markdown
Author

Closing in favor of #29040 which is more complete (also fixes the underscore escaping). Nice catch!

@FtlC-ian FtlC-ian closed this Feb 27, 2026
@jalehman
Copy link
Copy Markdown
Contributor

Thank you for alerting me to this though!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants