[Linter] Rule to use loadJsonContent instead json & loadTextContent
Open
story: linter rules
- Dominant language
- Bicep
- Stars
- 3.6k
- Forks
- 830
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 79
Description
Usage of `json(loadTextContent('jsonFile.json'))` should be discouraged by linter and we suggest converting code to use `loadJsonContent` function with a quick fix.
_Originally posted by @alex-frankel in https://github.com/Azure/bicep/issues/5846#issuecomment-1034150255_
To be done once #6563 is merged.
Contributor guide
Research direction
Start by reviewing prerequisite issue #6563, then locate the Bicep linter and quick-fix entry points. Confirm the existing handling of json(loadTextContent(...)) and identify the relevant tests before making changes. Done means the pattern is flagged and the quick fix converts it to loadJsonContent.
Written by the indexing model from the issue text.
Assessment
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100