WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#35899 closed enhancement (wontfix)

Clean up whitespace

Reported by: chris_dev Owned by:
Milestone: Priority: normal
Severity: minor Version: 4.6
Component: General Keywords: has-patch
Focuses: Cc:
PR Number:

Description

Remove some unneeded whitespace.

I'd like to start to contribute with really small one.

Attachments (1)

35899.diff (10.7 KB) - added by chris_dev 4 years ago.

Download all attachments as: .zip

Change History (3)

@chris_dev
4 years ago

#1 @chris_dev
4 years ago

  • Keywords has-patch added
  • Severity changed from normal to minor

#2 @netweb
4 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Hi @chris_dev welcome to Trac

Thanks for the ticket and the patch though whitespace changes are typically not accepted, when working on the code that surrounds these types of changes would then be updated/fixed. The reason being that when using SVN/Git blame we'd rather see the code change history than whitespace changes which makes debugging much easier.

"Code refactoring should not be done just because we can." /handbook/contribute/code-refactoring/

Thanks again but for closing this as wontfix.

Maybe take a look at some of the good first bug tickets and see if you'd be able to contribute via one these tickets

Last edited 4 years ago by netweb (previous) (diff)
Note: See TracTickets for help on using tickets.