-
Notifications
You must be signed in to change notification settings - Fork 13
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
fix an issue with prettify query #169
Conversation
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!
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.
Thanks!
@dmitryk-dk @Loori-R before merging this, do you think we should have a common approach for such issues like this and #167? Or #167 should be solved differently? |
I think we can merge both of the PRs, but in the follow-up commit, we can check if we can use only one function to handle different variables in one function like |
Fixed issue with prettify query if if the query includes Grafana variables in the lookbehind window.
See this issue: #166