Pikachu920

Why would it be replaced? `%entities% (is|are) (burning|ignited|on fire)` isn't even a expression anyway, it's a condition. If anything, this should be a new expression.

Well for one `is burning` would be needed for backwards compatibility, and it increases the englishness of skript. While you can do direct comparisons like that, for example `if damage...

> Moved it to `multiply` just to keep it with current standards of skript if we ever decide that `multiply` is outdated we can update it then I think lime...

I can't reproduce this on paper 1.19.2 using this script: ``` every second: give dirt to all players ``` I also tried the test script in the OP and couldn't...

honestly, I'm not sure if I like the syntax or concept . I think @FranKusmiruk's idea of `iteration count` is a better fit

>Remove the ability to use delete player's balance which doesn't make sense, users should use reset player's balance instead (though it's better to mention in release notes) ~~It is likely...

Seems like it may be an issue with the structure of the Event classes. `PlayerPortalEvent` (on portal) extends `PlayerTeleportEvent` (on teleport) extends `PlayerMoveEvent` (on any move). So basically, it seems...

I haven't tested but this check seem suspicious especially with the comment by it https://github.com/SkriptLang/Skript/blob/bc8458fccfc986f71ea4c36aaa02c64439d7f7e5/src/main/java/ch/njol/skript/SkriptEventHandler.java#L68-L69 This may also explain why the issue persists until you restart. It could be that...

I'm not sure I see the value here. Why not `expr.stream(e).map(converter::convert)`?

> > I'm not sure I see the value here. Why not `expr.stream(e).map(converter::convert)`? > > Way shorter, direct usage to lambdas, and nested expressions. Streams you also have to convert...