Here is one of the reasons why I got soft rejection.
Globals should always be within a function or a class and should be used restrictively & only if theme really needs to. It’s highly recommended not to use them at all just to keep things out of the global namespace, they’re poor coding practice.
I use redux framework. And in functions I call global to use its variables. e.g.
echo $redux_theme_field['logo']; // for example.
But how it must be? Thanks.