Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.x - Code Quality #2671

Merged
merged 9 commits into from
Apr 27, 2019
Merged

3.x - Code Quality #2671

merged 9 commits into from
Apr 27, 2019

Conversation

l0gicgate
Copy link
Member

This PR creates a cohesive docblock usage, import usage and removes the copyright headers from all files.

Closes #2666 and #2670

- Add missing `@throws`
  - Closes: slimphp#2666

- Format DocBlocks to be cohesive
  - Use imports instead of \ClassName
  - Line return after last `@param` and `@return`

  ```
  use Exception;

  /**
   * @param ...
   *
   * @return ...
   *
   * @throws Exception
   */
  ```

  Closes slimphp#2668
…xception class for some reason even though it's in the imports?!? Reverting back to previous code to see if that fixes it.
…xception class for some reason even though it's in the imports?!? COMPLETE REVERT TO SEE IF THIS FIXES THINGS
@l0gicgate l0gicgate added this to the 3.12.2 milestone Apr 27, 2019
@l0gicgate l0gicgate mentioned this pull request Apr 27, 2019
@coveralls
Copy link

coveralls commented Apr 27, 2019

Coverage Status

Coverage increased (+0.005%) to 97.318% when pulling 90beff6 on l0gicgate:3.x-CodeQuality into 09efe4d on slimphp:3.x.

@l0gicgate l0gicgate force-pushed the 3.x-CodeQuality branch 3 times, most recently from 4b47a13 to b7f0e5f Compare April 27, 2019 20:42
@l0gicgate l0gicgate merged commit e6f75e8 into slimphp:3.x Apr 27, 2019
@l0gicgate l0gicgate deleted the 3.x-CodeQuality branch May 8, 2019 05:19
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.

Slim\App docblocks have missing throws annotations
3 participants