Skip to content

Commit

Permalink
lint: better link to Crowdin
Browse files Browse the repository at this point in the history
A url was added in ad84757

The URL is the most important aspect, but it's inconveniently placed due
to the line length

So put it on a newline

```diff
- values-bn/01-core.xml:124: Error: should be 'JavaScript'; https://crowdin.com/editor/ankidroid/7290/en-bn#q=create_subdeck [TranslationTypo from com.ichi2.anki:lint-rules]
+ values-bn/01-core.xml:124: Error: should be 'JavaScript'
+ https://crowdin.com/editor/ankidroid/7290/en-bn#q=create_subdeck [TranslationTypo from com.ichi2.anki:lint-rules]
```
  • Loading branch information
david-allison committed Sep 30, 2024
1 parent 785d9e8 commit 67d698c
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ class TranslationTypo : ResourceXmlDetector(), XmlScanner {
fun Element.reportIssue(message: String) {
val elementToReport = this
val crowdinEditUrl = crowdinContext?.getEditUrl(elementToReport)
?.let { url -> "; $url" } ?: ""
?.let { url -> "\n$url" } ?: ""
context.report(
issue = ISSUE,
location = context.getElementLocation(elementToReport),
Expand Down

0 comments on commit 67d698c

Please sign in to comment.