Comments (2)
Two general approaches would be:
- Add an argument to the issues API that allows the rule to instruct on whether or not the range at issue is an expression or a declaration. This is likely to be a breaking change given that callers use
EmitIssue
to create issues and pass aRange
as its own argument. - Ignore reference parsing errors entirely:
Lines 324 to 327 in 5ed807d
On further thought it seems like 2 is actually ok given that this is meant to limit errors to expressions that reference module variables. If the expression is invalid it can't contain any variables.
from tflint.
Agreed with 2. 1 is a better approach, but I think module call inspection should stop assuming only issues for expressions in the first place.
In the future it would be good to redesign the plugin API by terraform-linters/tflint-plugin-sdk#193.
from tflint.
Related Issues (20)
- tflint in Docker not running the checks
- # tflint-ignore-file: terraform_required_version is being ignored
- Add support for Github auth token via environment variable HOT 1
- tflint does not recognise `removed` blocks
- Allow plugins located on github.com and GHES to be installed at once as an authenticated request HOT 2
- String interpolation passed as a list HOT 1
- Sarif output should use the final schema version HOT 2
- The latest version don't use the last version of the tflint-ruleset-azurerm
- Terraform 1.8.x builtin functions break tflint syntax parsing HOT 2
- Support monorepo release versions for plugins HOT 1
- Clarify license notices HOT 2
- BUSL-1.1 licensed code found in tflint HOT 2
- Allow use of `--chdir` and `--recursive` together HOT 1
- Add source Attribute to Checkstyle Format HOT 1
- Add python3.12 lambda runtime HOT 1
- tflint does not recognise ".tf" files at the root of the folder HOT 1
- Error when var file includes undeclared variables HOT 7
- Add support for Terraform v1.9
- Add support for evaluating static module outputs
- tflint --recursive --call-module-type=all cannot find nested local modules
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from tflint.