Results 113 comments of Joe Clack

I'm sorry you interpreted my tone as hostile, because that's not my intention. I'm just being matter-of-fact. I think your logic is flawed. From my understanding: - you tried some...

I can understand your logic, but it sounds to me like the issues you are experiencing are due to the *mods* you are using rather than BepInEx. When the game...

these are warnings. They have nothing to do with the problem you described. updating to BepInEx 5.4.22 will *not* fix the issue, just mute the warning.

> You might say that, but my ingame results speak for itself. Having an ingame audio play 9/10 times be fixed when I manually update BepInEx isn't just a warning....

> what is against updating the version of the mod in Thunderstore as well? Almost nothing. But there is also almost no reason to do so.

Any update on this ?

Let me know if you would like me to open a pull request adding the icon as a banner mirroring https://github.com/yt-dlp/yt-dlp, I'd be happy to!

Leaving this here for any future unfortunate souls - When working with `DefineConstants`, semicolons within its property value need to be escaped with `%3B`. The solution I proposed above will...

This does beg the question: Why are semicolons explicitly *not* escaped for `DefineConstants`? ```cs private static readonly HashSet _propertiesNotEscapeSemicolons = new HashSet { "DefineConstants", "ExcludeFilesFromDeployment" }; ```

I agree, the language isn't far off that rule - I think it's at around 120 - although it's unclear whether the listed rules are OR or AND