On Fri, 18 Oct 2024 08:39:49 GMT, Nizar Benalla <nbena...@openjdk.org> wrote:

>> Please review this change that adds a new test root `docs` dedicated to 
>> testing the documentation, which has been a work in progress for a while. 
>> Tests for links, encoding, HTML, accessibility will be later added in 
>> following PRs. 
>> 
>> We also define a new make target `test-docs` meant for local use and depends 
>> on the docs.
>> This also adds the necessary configurations needed at Oracle.
>> 
>> This patch includes a test `TestDocs` which serves to show developers how 
>> they are meant to resolve the docs to test them, I want to include it 
>> temporarily until better tests are added later.
>> 
>> TIA
>
> Nizar Benalla has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   use existing DOCS_JDK_IMAGE_DIR to be consistent.
>   
>   no ifneq check before assigning the env variable because  
> DOCS_JDK_IMAGE_DIR is set even docs do not exist. It will be up to the test 
> to check if the files are there

I'd like to avoid integrating this right before the weekend, and I still want 
to add copyright headers to the other TEST.ROOT files if I can find out when 
they were added. (I know how to do this)

-------------

PR Comment: https://git.openjdk.org/jdk/pull/21272#issuecomment-2422398300

Reply via email to