[jira] [Resolved] (HBASE-30455) Run website lint in precommit refguide check

["Junegunn Choi (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HBASE-30455?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Junegunn Choi resolved HBASE-30455.
-----------------------------------
    Fix Version/s: 4.0.0-alpha-1
       Resolution: Fixed

Thanks to [~nihaljain] and [~psomogyi] for the review!

> Run website lint in precommit refguide check
> --------------------------------------------
>
>                 Key: HBASE-30455
>                 URL: https://issues.apache.org/jira/browse/HBASE-30455
>             Project: HBase
>          Issue Type: Improvement
>          Components: build, website
>            Reporter: Junegunn Choi
>            Assignee: Junegunn Choi
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 4.0.0-alpha-1
>
>
> h2. Problem
> Precommit does not run {{npm run lint}} (ESLint and {{prettier --check}}) for {{hbase-website}}. Formatting errors pass the PR checks and only fail the post-merge {{HBase-Generate-Website}} job, which then stops publishing the website until someone pushes a fix.
> Since July 2026, four commits merged unformatted files and needed three follow-up fixes:
> - HBASE-30167: fixed by HBASE-30289
> - HBASE-30270 and HBASE-30220: fixed by the HBASE-30270 addendum
> - HBASE-30416: fixed by an addendum
> The precommit {{refguide}} check builds the site with {{-DskipTests}}. In {{hbase-website/pom.xml}}, {{skipTests}} activates the {{skip-tests}} profile, which selects the {{ci-skip-tests}} npm script instead of {{ci}}. {{ci-skip-tests}} does not include {{lint}}. The website job runs the full {{ci}} script, so it is the first place lint runs.
> h2. Fix
> Add {{npm run lint}} to {{ci-skip-tests}}, after {{fumadocs-init}}, the same position as in {{ci}}. Lint is not a test, so {{-DskipTests}} should not skip it. It takes about 5 seconds locally.
> No change is needed in {{hbase-personality.sh}}: a lint failure makes Maven print {{[ERROR]}}, and the {{refguide}} check already votes -1 on that.
> Also update {{hbase-website/README.md}}, which describes {{ci-skip-tests}} as running without lint.



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