HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly

[PDavid]

The author check fails in nightly build:

hbase-website/app/pages/_docs/docs/_mdx/(multi-page)/building-and-developing/developer-guidelines.mdx:154:Also, no `@author` tags - that's a rule.
hbase-website/public/book.html:40827:<p>Also, no <code>@author</code> tags - that’s a rule.</p>

https://ci-hbase.apache.org/job/HBase%20Nightly/job/master/1420/General_20Nightly_20Build_20Report/

Fix:
Ignore files under hbase-website in author checks.

[PDavid]

Hmm, it seems that the @author check is still failing in the general check. Does it checks it on master or on the branch? 🤔

[Apache9]

13:hbase-website/app/pages/_docs/docs/_mdx/(multi-page)/building-and-developing/developer-guidelines.mdx:154:Also, no `@author` tags - that's a rule.
14:hbase-website/public/book.html:40827:<p>Also, no <code>@author</code> tags - that’s a rule.</p>

Seems still the same error.

[Apache9]

This is the log console output, seems we have already used the new config?

AUTHOR_IGNORE_LIST=hbase-website/app/pages/_docs/docs/_mdx/(multi-page)/building-and-developing/developer-guidelines.mdx,hbase-website/public/book.html

[Apache9]

Oh wait a minute, what is the syntax for passing multiple ignore files in?

[Apache9]

Strange, it is comma separated... @ndimiduk do you have any ideas on this?

function author_parse_args
{
  declare i

  for i in "$@"; do
    case ${i} in
      --author-ignore-list=*)
        delete_parameter "${i}"
        yetus_comma_to_array AUTHOR_IGNORE_LIST "${i#*=}"
      ;;
    esac
  done
}

[PDavid]

Maybe we need space separator, let's see. 🙈

[ndimiduk]

https://issues.apache.org/jira/browse/YETUS-1266

[ndimiduk]

apache/yetus#366

[PDavid]

Many thanks @ndimiduk for your fix in Yetus! 👍

I guess we will have to wait until the new Yetus release and then upgrade Yetus in HBase, right?

[ndimiduk]

Many thanks @ndimiduk for your fix in Yetus! 👍

I guess we will have to wait until the new Yetus release and then upgrade Yetus in HBase, right?

Yes, unfortunately. Maybe you can re-implement the fix as a function override in our personality file -- bash is extremely monkey-patchable.