Simplify mix require

[JeffreyWay]

Note that I left your dang const as it is, even though it's stupid. let for life. 💃💃💃💃

[joshmanders]

Yo I'mma let you PR this, but const is the best keyword of all time. Of. All. Time.

[taylorotwell]

Thanks <3

[xargr]

@taylorotwell i think const is better solution than let becuse require is immutable.. perhaps...

[JeffreyWay]

const doesn't mean immutable. None of this matters even remotely.

…

On Tue, Jul 18, 2017 at 7:32 PM grigoris ***@***.***> wrote: @taylorotwell <https://github.com/taylorotwell> i think const is better solution than let becuse require is immutable.. perhaps... — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#4283 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AALLt5jOrvWmq9laK2qmO12ffQ1gYTCeks5sPUCZgaJpZM4NtS7M> .

[joshmanders]

const is visual debt, @xargr. Get out of our faces with that shizzle!

[xargr]

constant cannot change through re-assignment, and it can't be redeclared... i have saw over 10 tutorial for js framework and all developer uses const in require statement

[JeffreyWay]

I don't like const because it makes you assume that the value is immutable, when it's not.

https://www.dropbox.com/s/gnrz1r4nu0nkg5b/Screenshot%202017-07-19%2011.52.55.png?dl=0

It makes zero difference in this file. var would be equally fine.

[matthewlilley]

However, it does not allow the rebinding of that variable. Whereas let does.

The question here is "Does mix ever need to be rebound?"

[TuzBot]

I do agree with you @JeffreyWay but const does have slight immutability, at the very least in the example you have just attached.
Although you can change the value's of const person's properties you can't change person from an being Object when it was assigned, giving abit of clarity as to what type the variable is/intended/has to be to be used.

[JeffreyWay]

I think "slight immutability" is an oxymoron. :)

I use let for everything. Anyways, like I said it doesn't matter here. Zero people will reassign the mix variable.

[matthewlilley]

I think const should be default if no rebinding if needed. And aside from that, It's good to get into a habit of using the most appropriate variable keyword, more importantly, distinguishing the two.

[joshmanders]

I think bike-shedding is not really ideal, and if you don't want to use let just update it in your own webpack.mix.js file and be done with it.