-
Notifications
You must be signed in to change notification settings - Fork 124
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
Document cyclical error in include in UG #2198
Document cyclical error in include in UG #2198
Conversation
As this is a rare case (i.e., most users will not need to know about it), we should present it in a way most readers are not forced to read it but they can locate this information when they need to. Any ideas? |
Thank you @yucheng11122017 for taking this up. Could we:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, fixed a typo and added line numbers for consistency within the panel.
What is the purpose of this pull request?
Documents #2177
Overview of changes:
Added a warning about the cyclical error in includes when variables have the same name. Included suggested solution for this bug.
Anything you'd like to highlight/discuss:
Testing instructions:
Proposed commit message: (wrap lines at 72 characters)
Document cyclical error in include in UG
Checklist: ☑️