-
Notifications
You must be signed in to change notification settings - Fork 14
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.
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
Clean source dependency for readability #5
Comments
Yep. I agree, but it also means that you must add |
You have already added |
No, I meant for those not using cmake, like ESP32. I'll test it tonight and let you know when I'm done. |
Ah, I see! |
I think I've solved the issue, please test the latest master branch and report if it's ok for you. |
Thank you very much for your prompt response. I think what I pointed out has been properly corrected. So I close this issue. You don't need to merge #6 at all because it is problematic. |
I found some improvement for source readability during #4 .
is better than:
because
../../
is too strong method we can indicate any files without path setting.The text was updated successfully, but these errors were encountered: