WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#46499 closed enhancement (fixed)

Adds documentation to the abstract-testcase.php

Reported by: andizer's profile andizer Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.2 Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: has-patch
Focuses: docs Cc:

Description

Not all functions in abstract-testcase.php aren't documented properly. It would be nice to have these to provide some context.

Attachments (2)

46499.diff​ (11.1 KB) - added by andizer 6 years ago.
46499-2.diff​ (7.2 KB) - added by andizer 6 years ago.
Removed the @return void

Download all attachments as: .zip

Change History (7)

@andizer
6 years ago

#1 @SergeyBiryukov
6 years ago

  • Milestone changed from Awaiting Review to 5.2

#2 @SergeyBiryukov
6 years ago

Per the ​documentation standards, @return void should not be used outside of the default bundled themes. Looks good otherwise.

#3 @SergeyBiryukov
6 years ago

  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

@andizer
6 years ago

Removed the @return void

#4 @SergeyBiryukov
6 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 44902:

Docs: Improve documentation for phpunit/includes/abstract-testcase.php.

Props andizer.
Fixes #46499.

#5 @SergeyBiryukov
6 years ago

In 44904:

PHPCS: Remove extra space added in [44902].

See #46499.

Note: See TracTickets for help on using tickets.