-
Notifications
You must be signed in to change notification settings - Fork 355
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
tflint crashed #129
Comments
Hi @netflash, Thanks for your bug report.
Please use |
Thanks, @wata727.
|
Umm... It seems to be occurring when evaluating the attributes of a module. But I don't know how to reproduce it. |
@toddmichael hey mate, need some assistance here. |
@netflash, not sure which module, but I'm generally fine provided nothing sensitive (and there shouldn't be). Are we talking about just sharing for debugging purposes or do we need to open source the module and make public? Even there, this is something we're working toward so it's worth it to prioritize if it helps @wata727 help us. |
I had the same issue, I narrowed it down to where I'd set a variable to "default = 1" and then was using it as "${var.name}" with quotes. That would explain the "is int not string" error message. As soon as I changed the default to be "1" rather than 1 it didn't crash |
Oh... Thanks, @sidick. I noticed that I had made a very basic mistake. |
Problem sovled, thank you! |
Yep, fixed here too, thanks |
This is what I've got
version 0.4.0
not sure if I need to attach anything else to help debugging this
The text was updated successfully, but these errors were encountered: