Skip to content

Removed redundant @method tags & unused imports#498

Merged
greg-1-anderson merged 2 commits intoconsolidation:masterfrom
HypeMC:master
Jan 17, 2017
Merged

Removed redundant @method tags & unused imports#498
greg-1-anderson merged 2 commits intoconsolidation:masterfrom
HypeMC:master

Conversation

@HypeMC
Copy link

@HypeMC HypeMC commented Nov 28, 2016

Overview

This pull request:

  • Fixes a bug
  • Adds a feature
  • Breaks backwards compatibility
  • Has tests that cover changes

Summary

Removed redundant @method tags & unused imports.

Description

Most @method tags are redundant since, in the meantime, those methods were rewritten to not be magic ones. I've moved all the descriptions to the phpdoc for the actual methods. Also updated the @method tags on the FilesystemStack class to match the Symfony filesystem methods.

Removed all unused imports & those that tried to import classes that have been removed.

@coveralls
Copy link

coveralls commented Nov 28, 2016

Coverage Status

Coverage increased (+0.01%) to 58.504% when pulling f6b53df on Korikulum:master into aa55d66 on consolidation:master.

Korikulum added 2 commits November 28, 2016 19:48
…ods, moved descriptions to appropriate places.

Updated FilesystemStack @method tags to match those in Symfoyn Filesystem.
@coveralls
Copy link

coveralls commented Nov 28, 2016

Coverage Status

Coverage increased (+0.01%) to 58.504% when pulling 615f57e on Korikulum:master into aa55d66 on consolidation:master.

@greg-1-anderson
Copy link
Member

Thanks for this.

@greg-1-anderson greg-1-anderson merged commit b027871 into consolidation:master Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants