BeatSaber_UpdateSkipper icon indicating copy to clipboard operation
BeatSaber_UpdateSkipper copied to clipboard

Better logic for determining Beat Saber installation directory

Open VentyCZ opened this issue 2 years ago • 2 comments

This PR uses the following steps:

  1. Finds out the Steam installation directory from registry (32/64bit)
  2. Goes through all the defined library folders
  3. Finds the library folder containing Beat Saber and uses it

VentyCZ avatar Sep 08 '22 19:09 VentyCZ

Hey, thank you. I'll try to look into this in a bit

kinsi55 avatar Sep 11 '22 08:09 kinsi55

@kinsi55 Thank you :) Haven't done any C# in a long while, so even any nitpicks on the code are appreciated!

VentyCZ avatar Sep 11 '22 08:09 VentyCZ

Since you asked for it, I nitpicked :P

LGTM tho, I'll include it in the next update whenever that will be, thank you!

kinsi55 avatar Oct 09 '22 19:10 kinsi55

Np, thanks for the nitpicking too :)

Mostly code style stuff, except for using LINQ - haven't done any C# in looong while, so I totally forgot how to LINQ ;)

VentyCZ avatar Oct 09 '22 19:10 VentyCZ