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

Support Robo v1.X #61

Merged
merged 10 commits into from
May 7, 2017
Merged

Support Robo v1.X #61

merged 10 commits into from
May 7, 2017

Conversation

zdenekdrahos
Copy link
Member

@zdenekdrahos zdenekdrahos commented Apr 25, 2017

Based on #57, without dropping support for Robo v0.X

  • support robo v0 and v1
  • robo v1 upgrade
    • set logger in NonParallelExec and ParallelExec cause not setting logger is deprecated
    • use rawArg instead of arg in CodeAnalysisTask when setting values and options for tools
    • make declaration of QARunner::loadRoboFile() compatible with Robo\Runner::loadRoboFile($output)
    • exit code doesn't work - travis

Breaking The arg(), args() and option() methods in CommandArguments now escape the values passed in to them. There is now a rawArg() method if you need to add just one argument that has already been escaped.
ERROR: No logger set for Edge\QA\Task\ParallelExec. Use $this->task(Foo::class) rather than new Foo() in loadTasks to ensure the builder can initialize task the task, or use $this->collectionBuilder()->taskFoo() if creating one task from within another.
PHP Fatal error:  Uncaught Error: Call to undefined method Edge\QA\Task\NonParallelExec::getOutput() 

Tasks must use TaskIO for output methods. It is no longer possible to use IO from a task. For direct access use Robo::output() (not recommended).

https://github.com/EdgedesignCZ/phpqa/pull/57/files#diff-f9bcd99426b2eb566eabc2e8a09b18c2
ERROR: Declaration of QARunner::loadRoboFile() should be compatible with Robo\Runner::loadRoboFile($output)
@zdenekdrahos
Copy link
Member Author

@robertfausk: could you try this version? Thanks

composer update edgedesign/phpqa:dev-robo

@zdenekdrahos zdenekdrahos mentioned this pull request Apr 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant