Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#35899 closed enhancement (wontfix)

Clean up whitespace

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

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 11 years ago.

Download all attachments as: .zip

Change History (3)

@chris_dev
11 years ago

#1 @chris_dev
11 years ago

  • Keywords has-patch added
  • Severity normalminor

#2 @netweb
11 years ago

  • Milestone Awaiting Review
  • Resolutionwontfix
  • Status newclosed

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 11 years ago by netweb (previous) (diff)
Note: See TracTickets for help on using tickets.