Add Listener to PreAssetsCompileEvent

[WebMamba]

No description provided.

[WebMamba]

The PublicAssetsPathResolverInterface had some changes on the last release of AssetMapper. I'm not sure how we should handle it here. See the error on phpstan. Should we just remove the getPublicFilesystemPath from the SassPublicPathAssetPathResolver? What do you think @weaverryan?

[weaverryan]

I see it. Could we surround that with a method_exists? Would that do it?

public function getPublicFilesystemPath(): string
{
    if (!method_exists($this->decorator', 'getPublicFilesystemPath')) {
        throw new \Exception('...');
    }

    // rest of method
}

[WebMamba]

Damm didn't know about this trick to fool phpstan! Thanks! 😁

[WebMamba]

@bocharsky-bw I do not have the right to merge this PR, and I think this is ready to go 😁

[bocharsky-bw]

Sorry for the delay, I hoped for @weaverryan final review on it.

Thanks!

[ogizanagi]

The argument is missing:

            ->args([
                service('sass.builder')
            ])

[bocharsky-bw]

Sounds valid. Would you like to create a PR?

//cc @WebMamba