bitsondatadev commented on code in PR #9943:
URL: https://github.com/apache/iceberg/pull/9943#discussion_r1532832889


##########
site/README.md:
##########
@@ -60,7 +60,12 @@ The static Iceberg site pages are Markdown files that live 
at `/site/docs/*.md`.
 ```
 ### Building the versioned docs
 
-The Iceberg versioned docs are committed in the [orphan `docs` 
branch](https://github.com/apache/iceberg/tree/docs) and mounted using [git 
worktree](https://git-scm.com/docs/git-worktree) at build time. The `docs` 
branch contains the versioned documenation source files at the root. These 
versions are mounted at the `/site/docs/docs/<version>` directory at build 
time. The `latest` version, is a soft link to the most recent [semver 
version](https://semver.org/) in the `docs` branch. There is also an [orphan 
`javadoc` branch](https://github.com/apache/iceberg/tree/javadoc) that contains 
prior staticly generated versions of the javadocs mounted at 
`/site/docs/javadoc/<version>` during build time.
+The Iceberg versioned docs are committed in two 
[orphan](https://git-scm.com/docs/gitglossary#Documentation/gitglossary.txt-aiddeforphanaorphan)
 branches and mounted using [git 
worktree](https://git-scm.com/docs/git-worktree) at build time: 
+
+ 1. [`docs`](https://github.com/apache/iceberg/tree/docs) - contains the state 
of the documenation source files (`/docs`) during release. These versions are 
mounted at the `/site/docs/docs/<version>` directory at build time. 
+ 1. [`javadoc`](https://github.com/apache/iceberg/tree/javadoc) - contains 
prior statically generated versions of the javadocs mounted at 
`/site/docs/javadoc/<version>` directory at  build time.

Review Comment:
   > `1` -> `2`
   
   I prefer the markdown convention of using 1 all the time. This still 
[renders ascending 
numbers](https://github.com/bitsondatadev/iceberg/tree/add-nightly-local/site#building-the-versioned-docs)
 in the output, and doesn't require updates to numbered lists to be of concern 
during reviews and when updating the docs.



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to