unify warnings system
gerzytet opened this issue ยท 4 comments
Happy 200th issue!
Currently, we have 2 ways of warning users:
- The message that appears when an admin runs a trg command.
- The warnings system in the console.
Having both may be unnecessary.
After all, many users won't be affected by string interpolation changes, since they don't use $ in their strings. For them, seeing the message every time will get old fast. The warnings system only targets people who have an actual issue, so I think that it should be the only one.
currently, warnings only appear during parsing. There should only be a problem if for some reason a repeating trigger gets parsed every time it is activated
I didn't really go through every single part of the Deprecation Warning System, so I can't really tell which one would be a more suitable solution.
My only concern is that the Deprecation Warning System might spam the console if the Trigger is using the deprecated Executor, and the Trigger is some kind of repeatedly executed type(like Repeat trigger or using loops, etc.), then it ends up repeating the warnings as well.
I have no problem getting rid of option 1. if the spamming is not the case.