fix: Work around HCL's inability to interpret negative numbers #5038
Conversation
|
I'll take a look at this now... |
|
No more need, fixed: hashicorp/hil@dc4cc84 |
|
Thx, that was quick! 😃 |
|
No problem. For interest: I'm pretty sure this never worked and our PRs weren't testing properly before. There haven't been any changes to HIL since I extracted from Terraform which had no code changes in itself. No problem though, we support it now. :) |
|
Interesting. I spent several hours on Friday coming to this conclusion and was going to address it on Monday. Thanks! |
|
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
ghost
locked and limited conversation to collaborators
I recently merged #4854 which (to my surprise) caused one of the tests to fail.
The same test was passing ~11 days ago:
https://travis-ci.org/hashicorp/terraform/builds/105195588
I'm thinking that something must have changed in HCL parser.
This is more of a workaround to make tests green again, rather than a solution.