Skip to content
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.

Sign up for GitHub

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

Jump to bottom

We should add a fixture test that verifies that only strings or numbers can be values of NamedArgument #325

Open
zbraniecki opened this issue Feb 7, 2021 · 1 comment
Open

We should add a fixture test that verifies that only strings or numbers can be values of NamedArgument #325

zbraniecki opened this issue Feb 7, 2021 · 1 comment

Comments

@zbraniecki
Copy link
Collaborator

I just discovered that fluent-rs allows for any inline expression to be a value of a NamedArgument and no fixture checks that.
@rkh
Copy link

rkh commented Feb 24, 2021

Not sure this is the right place to ask, but what is the rational for only supporting strings and numbers as named arguments?

@rkh rkh mentioned this issue May 12, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rkh @zbraniecki