👎 This would be a breaking change for projects that don't define this in their own configs. IMHO framework level config changes should be done on major versions only

Sure, just doing as requested. Initial contribution was changing from Str::slug(..., '_') to Str::snake(...) which wasn't a breaking change.

The original was changing the boilerplate that does not effect existing apps, this is changing the framework default configs that will if these are not already defined in the env, this resulting in a huge breaking change especially for the cookie change. There's 0% this will go through without issues.

I guess they'll have to be left out of sync then and updated later on.

@axlon, @donnysim - retarget to master or not worth it?

I think we need to target the master branch.

@axlon, @donnysim - retarget to master or not worth it?

I haven't read up on why this change is being made, but if it is the default on the boilerplate now it probably makes sense to PR it to master here (and should be added to the upgrade guide for 13.x)

Super - we got there in the end