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

DX: Include symfony/var-dumper in dev tools #7795

Merged
merged 3 commits into from Feb 3, 2024

Conversation

Wirone
Copy link
Member

@Wirone Wirone commented Feb 1, 2024

Adding require __DIR__.'/dev-tools/autoload.php'; to .php-cs-fixer.php can help during development because it makes dump() and dd() available. In many cases XDebug is just better, but sometimes simple dump can really help.

@Wirone Wirone requested a review from keradus February 1, 2024 13:09
@Wirone Wirone self-assigned this Feb 1, 2024
@coveralls
Copy link

coveralls commented Feb 1, 2024

Coverage Status

coverage: 94.753%. remained the same
when pulling 425377b on Wirone:codito/symfony-var-dumper
into dfa0084 on PHP-CS-Fixer:master.

dev-tools/composer.json Outdated Show resolved Hide resolved
@keradus keradus requested review from keradus and removed request for keradus February 1, 2024 23:59
Adding `require __DIR__.'/dev-tools/autoload.php';` to `.php-cs-fixer.php` can help during development because it makes `dump()` and `dd()` available. In many cases XDebug is just better, but sometimes simple dump can really help.
@keradus keradus removed their request for review February 2, 2024 23:28
@keradus
Copy link
Member

keradus commented Feb 2, 2024

(I put myself out of review, as - like previously mentioned - no opinion on my side, I'm not using this library ever (at least directly and willingly))

@keradus
Copy link
Member

keradus commented Feb 2, 2024

maybe @julienfalque @kubawerlos can help to review? :)

@keradus keradus enabled auto-merge (squash) February 2, 2024 23:29
@Wirone
Copy link
Member Author

Wirone commented Feb 3, 2024

(I put myself out of review, as - like previously mentioned - no opinion on my side, I'm not using this library ever (at least directly and willingly))

Are you trying to tell you never did Dump Driven Development 😆?

@keradus keradus merged commit d15cfea into PHP-CS-Fixer:master Feb 3, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants