chore: support multi tag value selection

influxdata / flux-lsp

Implementation of Language Server Protocol for the flux language

MIT License

27 stars 5 forks source link

Closed appletreeisyellow closed 1 year ago

appletreeisyellow commented 1 year ago

Close influxdata/ui#5949

This PR updates the AST for multi tag values in the following rules:

If adding a tag value where the tag key already exists in the flux file, use or.
If adding a tag value with a new tag key, append a new filter(...) function. This function as implicit and.

appletreeisyellow commented 1 year ago

Good question! I'll add test cases for queries with and in the filter() function and see the result

appletreeisyellow commented 1 year ago

I'm curious to know if this breaks the sync for existing queries

@asalem1 I added more test cases to ensure this change doesn't break the existing sync