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: confInfo set higher priority filepath #165

Open

oyk666 wants to merge 3 commits into go-chassis:master from oyk666:bugfix/fix_config_filepath

oyk666 commented Mar 30, 2024

if we add filename3 before filename1 and filename2, then the test failed.

cmSource.AddFile(filename3, 2, nil)
cmSource.AddFile(filename1, 0, nil)
cmSource.AddFile(filename2, 1, nil)

This is because we did not save the corresponding filepath when encountering a higher priority value.

OUYANGKAI added 3 commits

March 30, 2024 12:53


          fix: confInfo set higher priority filepath

7702d8e


          fix: if confInfo == nil, confInfo.Value = nil will panic

81943d3


          remove for range statement to reduce time spend

ff55903

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

Labels

None yet