New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix blank external config bug #46

Merged

apexatoll merged 2 commits into main from fix-blank-external-config-bug

Nov 9, 2023

Owner

apexatoll commented Nov 9, 2023

This PR fixes a bug where a target application that sets an external config file, but the file is blank, will raise an error when applying application config
This is due to YAML.load_file unexpectedly returning nil rather than an empty hash when the file is empty

apexatoll added 2 commits

November 9, 2023 20:33


          Extend config specs to handle empty external config file

e25d3b7


          Fix OpenConfigurator to handle blank yaml files

apexatoll force-pushed the fix-blank-external-config-bug branch from ca6b7ec to 7653422 Compare

November 9, 2023 20:33

apexatoll merged commit d11de1a into main

apexatoll deleted the fix-blank-external-config-bug branch

November 9, 2023 20:36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet