Error context for string interpolation may be wrong
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 20.6k
- Forks
- 3.3k
- PR merge metrics
- PR metrics pending
Description
If there are multiple replacements like '%s, %s' % (one, other), many errors (e.g. wrong replacement type) are reported with the tuple expression as the location. It would be better to set specific item as the error context as the current way can be quite misleading if one uses, --show-column-numbers, or --pretty, or one has a multi-line interpolation.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the multi-replacement interpolation case and compare the reported location with and without --show-column-numbers and --pretty, including a multi-line expression. Trace the interpolation error-reporting path and add coverage showing that each replacement error points to its specific item rather than the tuple expression.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- compilers
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100