Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
Apache9 commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4272933461 We can try to use the lastest master of yetus to see if it helps? If so, we can push yetus to release a new version. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
PDavid commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4143063731 Hi @ndimiduk, I tried to do the monkey-patching thing but to be honest I'm not sure if I'm doing it right. It does not work for sure. I cannot see my newly added debug messages in the output log. :see_no_evil: Can you please help in this a bit? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
ndimiduk commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4116963841 > 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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
PDavid commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4116853072 Many thanks @ndimiduk for your fix in Yetus! :+1: I guess we will have to wait until the new Yetus release and then upgrade Yetus in HBase, right? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
ndimiduk commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4097707636 https://github.com/apache/yetus/pull/366 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
ndimiduk commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4097554446 https://issues.apache.org/jira/browse/YETUS-1266 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
PDavid commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4074343522 Maybe we need space separator, let's see. :see_no_evil: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
Apache9 commented on PR #7952:
URL: https://github.com/apache/hbase/pull/7952#issuecomment-4074324764
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
}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
Apache9 commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4074297786 Oh wait a minute, what is the syntax for passing multiple ignore files in? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
Apache9 commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4074291105 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 ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
Apache9 commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4074288402 ``` 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:Also, no @author tags - that’s a rule. ``` Seems still the same error. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-29878 Improve the Documentation UI (addendum) - fix author check in nightly [hbase]
PDavid commented on PR #7952: URL: https://github.com/apache/hbase/pull/7952#issuecomment-4074169614 Hmm, it seems that the `@author` check is still failing in the general check. Does it checks it on master or on the branch? :thinking: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
