[DiscordArchive] That pr was tested locally before it got merged right?
[DiscordArchive] That pr was tested locally before it got merged right?
Archived author: M'Dic • Posted: 2023-01-21T18:07:57.394000+00:00
Original source
That pr was tested locally before it got merged right?
Archived author: Bench • Posted: 2023-01-21T18:18:21.471000+00:00
Original source
Yep, all the warnings were intentional and were noted on Github before it got merged iirc
Archived author: M'Dic • Posted: 2023-01-21T18:20:38.534000+00:00
Original source
So folks just need to fix their confs then
Archived author: Bench • Posted: 2023-01-21T18:20:45.050000+00:00
Original source
Pretty much
Archived author: Bench • Posted: 2023-01-21T18:20:48.366000+00:00
Original source
Or just ignore the red
Archived author: Honey • Posted: 2023-01-21T18:22:01.410000+00:00
Original source
Looks like a good occasion to do that. New confs were consequently skipped and left at default before.
Archived author: Foe • Posted: 2023-01-21T18:22:20.648000+00:00
Original source
It will give you a warning every time the console tries to read a config value from the config file that doesn't exist
Archived author: Foe • Posted: 2023-01-21T18:23:07.899000+00:00
Original source
I would highly recommend rewriting the config load system to pre-load all config values before they are attempted to be used in the core, but that's beyond the scope of removing dist file dependency
Archived author: Foe • Posted: 2023-01-21T18:23:19.433000+00:00
Original source
Yes, it was
Archived author: Kitzunu • Posted: 2023-01-21T18:23:58.201000+00:00
Original source
this would be good