Close #31: Add option to auto-update game

[Gymnasiast]

No description provided.

[733737]

I posted a few bugs in the discord.

[733737]

when you have a development build installed and automatically install updates checked, then you uncheck show development versions, it downloads the release build but does not install the release build. So you are left with the development build until you uncheck automatically install updates and click download.

[Gymnasiast]

@733737 Is that the only remaining bug? (If any of the things you mentioned on Discord are still applicable, please mention them as well.)

[733737]

I don't like how automatic downloads are universal. Also, it instantly downloads the latest build when you click off to openloco and back so if it could detect a new build a little faster that would be good.

[IntelOrca]

Remove async from autoUpdateGameCheckbox_Changed too to get rid of the warnings.

Typically methods that are async should return Task instead of void. But UI events must return void and therefore can't do this. Instead we wrap everything in a try catch so that we can handle exceptions if they get raised.

Since the DownloadBuild method does wrap everything in a try catch, this can remain a return void function, and the events don't need to await it and can just be non-async void themselves.

[733737]

all of the bugs I reported seem to be fixed! 👍

[Manticore-007]

this doesn't look good, the weird symbols at the start of the messages I mean.

[Gymnasiast]

Those are colour codes to make the text in the terminal look nice. It’s not related to this PR anyway - it will also happen if you install OpenLoco manually and then try to start it without Locomotion installed.