Skip to content

Add support for baking a Command Class#449

Merged
markstory merged 2 commits intocakephp:masterfrom
Iandenh:command
May 26, 2018
Merged

Add support for baking a Command Class#449
markstory merged 2 commits intocakephp:masterfrom
Iandenh:command

Conversation

@Iandenh
Copy link
Copy Markdown
Contributor

@Iandenh Iandenh commented May 25, 2018

Make it possible to bake a Command class.

bake command Example

}

if ($type !== 'Controller') {
if ($type !== 'Controller' && $type !== 'Command' ) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expected 0 spaces before closing bracket; 1 found

}


/**
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found more than a single empty line between content

@codecov
Copy link
Copy Markdown

codecov bot commented May 25, 2018

Codecov Report

Merging #449 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #449      +/-   ##
============================================
+ Coverage     94.25%   94.27%   +0.02%     
- Complexity      685      689       +4     
============================================
  Files            26       27       +1     
  Lines          2192     2200       +8     
============================================
+ Hits           2066     2074       +8     
  Misses          126      126
Impacted Files Coverage Δ Complexity Δ
src/Shell/Task/CommandTask.php 100% <100%> (ø) 3 <3> (?)
src/Shell/Task/TestTask.php 97.56% <100%> (+0.01%) 90 <0> (+1) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2daf5b0...6bfffde. Read the comment docs.

@markstory markstory added this to the 1.8.0 milestone May 25, 2018
@markstory markstory self-assigned this May 25, 2018
@markstory markstory merged commit 1e54dd0 into cakephp:master May 26, 2018
@Iandenh Iandenh deleted the command branch May 26, 2018 10:30
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