-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
WIP: Kickoff #1
WIP: Kickoff #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some tiny things ;-)
composer.json
Outdated
@@ -0,0 +1,44 @@ | |||
{ | |||
"name": "apitte/console", | |||
"minimum-stability": "dev", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please reformat composer.json according to the others. (https://github.com/apitte/openapi/blob/master/composer.json)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove composer.lock.
composer.json
Outdated
"prefer-stable": true, | ||
"require": { | ||
"php": ">= 5.6", | ||
"apitte/core": "^0.1.0@dev", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Depend on stable version.
src/Command/RouteDumpCommand.php
Outdated
use Symfony\Component\Console\Style\SymfonyStyle; | ||
|
||
/** | ||
* Class RouteDumpCommand |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this phpdoc - it's useles, the name is already in filename.
src/Command/RouteDumpCommand.php
Outdated
const TABLE_HEADER = ['Method', 'Path', 'Handler', 'Parameters']; | ||
|
||
/** | ||
* @var \Apitte\Core\Schema\Schema |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use single link phpdoc and import namespace for Schema;
/** | ||
* @return void | ||
*/ | ||
protected function configure() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The first method should be construct.
src/Command/RouteDumpCommand.php
Outdated
} | ||
|
||
/** | ||
* RouteDumpCommand constructor. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this phpdoc except the $schema.
src/Command/RouteDumpCommand.php
Outdated
* @param \Symfony\Component\Console\Output\OutputInterface $output | ||
* @return void | ||
*/ | ||
public function printEndpointsTable(OutputInterface $output) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move this to execute method please.
Thank you. |
Hi,
this PR is just kickoff for making this package ready. Can we start working on this? I just need to know what needs to be done and I will update this PR.
Thanks