Piotrek

Hello @tfgrass thanks for creating the PR. Any reason why not to use `WP_ROCKET_CACHE_ROOT_PATH` constant?

@tfgrass I might be missing something, but for me it doesn't make sense to define `WP_ROCKET_CACHE_PATH` like this. Mentioned constant is dependent on the `WP_ROCKET_CACHE_ROOT_PATH`, so if you set `WP_ROCKET_CACHE_ROOT_PATH`...

Also, `inc/front/process.php` is not used since many many years

@jorgeatorres Would it be possible for you to give us any hint about where to start with implementation? We'd be happy to contribute to the code here.

@Tabrisrp What's your opinion on that?

@MathieuLamiot Yes, that's acceptable while taking into the consideration complexity. Let's deliver this one and create a low priority issue after the release about the missing class.

@jeawhanlee Where are we at with this PR?

@Mai-Saad That's fine with the order

@CrochetFeve0251 @Tabrisrp Apologies, thought that the grooming will happen. I added this issue to the backen repository, so we could move this part there: https://github.com/wp-media/wp-rocket/blob/3abb772b945c7154348a44a0e6d13e51f64c9a2c/inc/Engine/Optimization/Minify/JS/AbstractJSOptimization.php#L100-L104

Thanks for creating the issue. @wp-media/engineering-team What do you think about this request?