E501: strings links should allow “introduction”
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 5.2k
- Forks
- 754
- PR merge metrics
- No merged PRs in 30d
Description
The line length checker excepts URL only lines because of #224 which is sensible. But usually there is some introduction like:
So maybe there is a way to allow something before an URL when it's a comment or doc string. The question is now what is sensible. Maybe something like [^ ]+: so it would allow the above line or something like:
See: https://example.com/a-very-very-very…
It wouldn't allow “See also:” so maybe it should just strip the URL and check the rest of the line?
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 with the E501 implementation and the behavior introduced by #224. Resolve whether comment or docstring lines with introductory text should be accepted, define the exact rule, and verify it against the URL examples in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100