[ 
https://issues.apache.org/jira/browse/NUTCH-3216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18126885#comment-18126885
 ] 

ASF GitHub Bot commented on NUTCH-3216:
---------------------------------------

lewismc merged PR #978:
URL: https://github.com/apache/nutch/pull/978




> Fix remaining codespell findings flagged by Yetus on master
> -----------------------------------------------------------
>
>                 Key: NUTCH-3216
>                 URL: https://issues.apache.org/jira/browse/NUTCH-3216
>             Project: Nutch
>          Issue Type: Improvement
>    Affects Versions: 1.23
>            Reporter: Lewis John McGibbney
>            Assignee: Lewis John McGibbney
>            Priority: Minor
>             Fix For: 1.24
>
>
> Master Yetus run 36915201085 reports 20 codespell issues 
> ({{{}results-codespell.txt{}}}, codespell 2.4.1). Thirteen are real typos in 
> tests and one fixture:
>  * {{bage}} in {{src/testresources/fetch-test-site/pageb.html}}
>  * {{readed}} in {{TestMetadata}} and {{TestSpellCheckedMetadata}}
>  * {{atLeast}} in {{TestErrorTracker}}
>  * {{compressable}} in {{TestGZIPUtils}}
>  * {{presse}} in {{TestURLUtil}}
>  * {{instanciated}} in 
> {{src/test/org/apache/nutch/parse/parse-plugin-test.xml}}
> Seven are language names in 
> {{src/plugin/language-identifier/src/java/org/apache/nutch/analysis/lang/langmappings.properties}}
>  ({{{}wel{}}}, {{{}nd{}}}, {{{}sme{}}}, {{{}som{}}}, {{{}te{}}}, {{{}tha{}}}, 
> {{{}vie{}}}). That path is already listed in {{.codespellrc}} {{{}skip{}}}, 
> and it is not listed in {{.yetus/excludes.txt}} (that file excludes the 
> language-identifier test corpora only). The master run still reported the 
> properties file, so confirm whether the Yetus codespell plugin honors 
> {{skip}} before editing it. Fix the real typos. Do not rewrite language-name 
> tokens if the skip can be made effective.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to