Skip to content

Fix missing DIContainerTrait when StaticAddToTrait is used#1251

Merged
mvorisek merged 1 commit intoatk4:developfrom
mvorisek:fix_for_new_factory
Jun 7, 2020
Merged

Fix missing DIContainerTrait when StaticAddToTrait is used#1251
mvorisek merged 1 commit intoatk4:developfrom
mvorisek:fix_for_new_factory

Conversation

@mvorisek
Copy link
Copy Markdown
Member

@mvorisek mvorisek commented Jun 7, 2020

Refactor code for atk4/core#221

fully documented behaviour that DIContainerTrait trait much be used when StaticAddToTrait is used

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 7, 2020

Codecov Report

Merging #1251 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop    #1251   +/-   ##
==========================================
  Coverage      72.35%   72.35%           
  Complexity      2556     2556           
==========================================
  Files            130      130           
  Lines           6313     6313           
==========================================
  Hits            4568     4568           
  Misses          1745     1745           
Impacted Files Coverage Δ Complexity Δ
src/Template.php 92.11% <ø> (ø) 101.00 <0.00> (ø)

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 19e31a1...a83dc81. Read the comment docs.

@mvorisek mvorisek requested a review from DarkSide666 June 7, 2020 11:23
@mvorisek mvorisek closed this Jun 7, 2020
@mvorisek mvorisek deleted the fix_for_new_factory branch June 7, 2020 12:17
@mvorisek mvorisek restored the fix_for_new_factory branch June 7, 2020 12:51
Copy link
Copy Markdown
Member

@romaninsh romaninsh left a comment

Choose a reason for hiding this comment

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

LGTM

@mvorisek mvorisek merged commit 23067a5 into atk4:develop Jun 7, 2020
@mvorisek mvorisek deleted the fix_for_new_factory branch June 7, 2020 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants