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.

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

Seems like some issues with inline variables #1

Open
jcward opened this issue Jul 18, 2018 · 2 comments
Open

Seems like some issues with inline variables #1

jcward opened this issue Jul 18, 2018 · 2 comments

Comments

@jcward
Copy link

jcward commented Jul 18, 2018

image

Maybe remove the inline? Or maybe don't instrument these?

To workaround, I just removed the inline myself.

@nadako
Copy link
Owner

nadako commented Jul 18, 2018

i think they just shouldn't be instrumented, since the value of the field is never actually used

@jcward
Copy link
Author

jcward commented Jul 18, 2018

FYI, I've pushed the suite with coverage.hxml into my lib repo. If you pull it (next to ./coverme/), you should be able to run haxe coverage.hxml (may have to haxelib install buddy) for testing.

https://github.com/jcward/haxe-graphql/tree/master/test/buddy
jcward/haxe-graphql@cd1494a88e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants