Closed weitzman closed 1 year ago
Merging #109 (0555735) into 4.x (33e140d) will increase coverage by
0.04%
. The diff coverage is100.00%
.:exclamation: Current head 0555735 differs from pull request most recent head 060da92. Consider uploading reports for the commit 060da92 to get more accurate results
@@ Coverage Diff @@
## 4.x #109 +/- ##
============================================
+ Coverage 91.88% 91.93% +0.04%
- Complexity 526 530 +4
============================================
Files 50 50
Lines 1318 1326 +8
============================================
+ Hits 1211 1219 +8
Misses 107 107
Impacted Files | Coverage Δ | |
---|---|---|
src/Formatters/VarDumpFormatter.php | 92.30% <100.00%> (+3.41%) |
:arrow_up: |
src/Formatters/VarExportFormatter.php | 100.00% <100.00%> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Also add simplifier for Drupal Entities
This works with this this PR and not without:
drush ev 'return \Drupal\user\Entity\User::load(1)' --format=yaml
drush ev 'return \Drupal::keyValue("entity.definitions.installed")->get("user.entity_type")' --format=var_dump
Once this PR goes in, IMO we should change the Drush's default format in php:eval from var_export to var_dump (since it is colored and thus easier to read). Also we should bump the min version of consolidation/output-formatters
Overview
This pull request: