grafana / sqlds

A package that assists writing SQL-driven datasources
Apache License 2.0
20 stars 12 forks source link

Macros/Multi param function args #98

Closed bossinc closed 1 year ago

bossinc commented 1 year ago

Functions passed into macros that have multiple args will parse correctly.

Before if your macro was select * from foo where $__params(FUNC(foo, bar)) it would parse the macro params as ["FUNC(foo", "bar)"] It will now parse the params as ["FUNC(foo, bar)"]

I also created a Macro struct to make it clear which part is the macro name and args.

CLAassistant commented 1 year ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant commented 1 year ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.