Can we get some context here? I am not up-to-speed on my npm cli knowledge, but last I knew you had to pass -- to denote that the flags after should be passed along. Has this changed?

The -- is related to #4196

Oops. Sorry, my bad. @joshmanders right. https://docs.npmjs.com/cli/run-script

Should default to yarn anyway.

Just a question after watching some MRs in this repo, and I notice a lot of them are missing some kind of justification or explanation of their changes, and still gets merged. Is there some internal chat of some sort where this is done, shouldn't there be more explanation here why the lines changed by this MR was "not needed"?

Please ignore this, it is my fault, I will be more careful in the next PR opening. And I will write descriptions, not just short ones. My bad.
:(

IMHO any PR regardless if it's the next greatest thing to come to the framework should be closed without a proper title and description.

It is quite easy to see what code is and does with the changes tab, the creator of laravel also uses this simplistic titles as this does if you look through the history of commits.

@pierreneter No worries, mistakes happen to the all of us. 😄