Closed meliache closed 11 months ago
Attention: 5 lines
in your changes are missing coverage. Please review.
Comparison is base (
5815d46
) 60.51% compared to head (b8c890a
) 60.87%. Report is 9 commits behind head on main.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Changed the default value of the
gbasf2_install_directory
setting and functions which take this as a parameter to the new CVMFS setup script path in gbasf2 v5.8.2. Also fixed the documentation for the setting.The old setting
gbasf2_install_directory
has been fully deprecated now. Before I tried to use the old setting as a fallback ifgbasf2_setup_path
was not customized butgbasf2_setup_path
was. But that resulted in some complicated error-prone logic which is annoying to update when gbasf2 changes their setup path. However I still don't raise an exception when the old setting is used and just raise a warning and fall back to the value ofgbas2_install_directory
. Most users just used the default anyway.The changes are also documented in the updated changelog.
This solves #206 reported by @miverone and @0ctagon. Please test if this solves the
FileNotFoundError
at least.The issue comments contain potential other bugs with the new gbasf2 release, but they seem like they should be a separate more complex issue and I'd like to merge this simple path fix first.
Resolves #206