search

digitalbazaar / eslint-config-digitalbazaar

BSD 3-Clause "New" or "Revised" License
2 stars 1 forks source link

Add `ignoreUrls: true` to `max-len` rule. #77

Closed BigBlueHat closed 4 months ago

BigBlueHat commented 4 months ago

Long URLs are increasingly common in our test suite code since we are using text selection fragments to connect to specific highlights in specifications.

davidlehn commented 4 months ago

The regex is aimed at URLs with :// in them. Good enough for now unless we need it for long did:s or similar. https://github.com/eslint/eslint/blob/284722ca8375c9a9e4f741bfdd78e765542da61f/lib/rules/max-len.js#L101