[
https://issues.apache.org/jira/browse/NUTCH-3197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18105134#comment-18105134
]
ASF GitHub Bot commented on NUTCH-3197:
---------------------------------------
lewismc opened a new pull request, #956:
URL: https://github.com/apache/nutch/pull/956
PR for [NUTCH-3197](https://issues.apache.org/jira/browse/NUTCH-3197)
Follow-up to [NUTCH-3196](https://github.com/apache/nutch/pull/947) (Child
A: Yetus false-positive baselines). This is **Child B** of the Yetus master
precommit initiative: fix real defects the gate correctly reported, without
taking on historical style debt (Child C) or shellcheck warnings/notes (Child
D).
## Summary
- Fix 11 shellcheck **errors** in `src/bin/crawl` and `src/bin/nutch`
(SC2071, SC2144, SC2242, SC2145) with behavior-preserving edits
- Add `@audience` / `@stability` shelldocs annotations to five helper
functions in `src/bin/crawl`
- Restructure `docker/Dockerfile` for hadolint (combined RUNs, quoting);
waive DL3018 via `.hadolint.yaml` (no floating apk pin policy)
- Exclude `src/plugin/parse-js/sample/` from jshint via
`.yetus/excludes.txt` (test fixture, not production JS)
Product **codespell** typos are intentionally deferred to a linked follow-up
PR under the same Jira so this diff stays reviewable.
## Out of scope (sibling issues)
- Historical blanks/tabs, `@author`, markdownlint → Child C (NUTCH-3198)
- Shellcheck warnings/notes in bin scripts → Child D (NUTCH-3199)
- Product codespell typos → linked PR2 under NUTCH-3197
## Manual smoke checklist
- [x] `runtime/local/bin/nutch` — prints command list
- [x] `runtime/local/bin/nutch parsechecker` — prints help
- [x] `runtime/local/bin/crawl` — prints usage on missing/invalid args;
exits with status 1 (not 255)
- [x] `shellcheck -S error src/bin/crawl src/bin/nutch` — no errors
## Test plan
- [x] `ant clean runtime test`
- [ ] Yetus CI on this PR clears shellcheck-error, shelldocs, hadolint,
jshint subsystems
- [ ] Confirm full-tree master may still fail on style debt — expected until
C/D land
Made with [Cursor](https://cursor.com)
> Yetus: fix the real problems the precommit gate correctly reported
> ------------------------------------------------------------------
>
> Key: NUTCH-3197
> URL: https://issues.apache.org/jira/browse/NUTCH-3197
> Project: Nutch
> Issue Type: Sub-task
> Reporter: Lewis John McGibbney
> Priority: Major
>
> After baselines, the remaining interesting failures are things we would want
> fixed even without Yetus: misspellings in Java/docs that are not deliberate
> test inputs; eleven shellcheck *errors* in src/bin/crawl and src/bin/nutch
> (bad exit usage, glob/-f mistakes, string-vs-numeric compare, "$@"
> mishandling); Dockerfile structure issues; missing @audience/@stability
> shelldocs on crawl helpers; and a single jshint style nit in the parse-js
> sample.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
