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

[Insight for Laravel] Verify if there no dd() calls #12

Closed
nunomaduro opened this issue Apr 9, 2019 · 4 comments · Fixed by #32
Closed

[Insight for Laravel] Verify if there no dd() calls #12

nunomaduro opened this issue Apr 9, 2019 · 4 comments · Fixed by #32

Comments

@nunomaduro
Copy link
Owner

nunomaduro commented Apr 9, 2019

To solve this issue, you should create a pull request adding the Insight described on the title.

@owenvoke
Copy link
Contributor

owenvoke commented May 8, 2019

Is there a way to view function call names at the moment? 🤔

@szepeviktor
Copy link
Contributor

it is Squiz.PHP.DiscouragedFunctions.forbiddenFunctions in PHPCS

Could we add dump() and extract, compact and list?

@nunomaduro
Copy link
Owner Author

Exactly, good idea @szepeviktor.

@pxgamer You just need to add this code:

                ForbiddenFunctionsSniff::class => [
                    'sizeof' => 'count',
                    'delete' => 'unset',
                    // Any other...
                ]

To

.

Don't forget to test it locally to see if it works.

@owenvoke
Copy link
Contributor

owenvoke commented May 8, 2019

I'll take a look at it later, cheers. 👍

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 a pull request may close this issue.

3 participants