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

Hudson commented on NUTCH-3216:
-------------------------------

SUCCESS: Integrated in Jenkins build Nutch ยป Nutch-trunk #277 (See 
[https://ci-builds.apache.org/job/Nutch/job/Nutch-trunk/277/])
NUTCH-3216 Fix remaining codespell findings flagged by Yetus on master (#978) 
(github: 
[https://github.com/apache/nutch/commit/f1610723f016902628427e8361b8e0dd9128d794])
* (edit) src/test/org/apache/nutch/metadata/TestMetadata.java
* (edit) src/testresources/fetch-test-site/pageb.html
* (edit) src/test/org/apache/nutch/metadata/TestSpellCheckedMetadata.java
* (edit) src/test/org/apache/nutch/parse/parse-plugin-test.xml
* (edit) .codespellrc
* (edit) src/test/org/apache/nutch/util/TestGZIPUtils.java
* (edit) src/test/org/apache/nutch/util/TestURLUtil.java


> 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: Sub-task
>    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