Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

lidavidm commented on PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#issuecomment-2608784051 https://github.com/apache/iceberg-cpp/issues/36 :sweat_smile: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

lidavidm commented on PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#issuecomment-2608783393 I am not sure what I was doing there, good catch. Let me fix that. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

wgtmac commented on PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#issuecomment-2608781211 > Thanks, I filed [apache/arrow-java#555](https://github.com/apache/arrow-java/issues/555) to keep track of that. It seems that the issue was created against the wrong repo... --

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

lidavidm commented on PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#issuecomment-2608767701 Thanks, I filed https://github.com/apache/arrow-java/issues/555 to keep track of that. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

Fokko commented on PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#issuecomment-2595752919 Thanks everyone, let's move this forward 👍 -- 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 t

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

Fokko commented on PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#issuecomment-2595760514 Once we get the build up and running and publish the static pages to the `gh-pages` branch, I'm happy to help to get `cpp.iceberg.apache.org` up and running 👍 -- This is an automate

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

Fokko merged PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27 -- 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.apach

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

Fokko commented on code in PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#discussion_r1917817748 ## docs/README.md: ## @@ -0,0 +1,29 @@ + + +# Documentation + +To build the documentation: + +#. Install [Doxygen][doxygen]. +#. From this directory, run `doxygen`. Revi

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

Fokko commented on code in PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#discussion_r1917816727 ## docs/README.md: ## @@ -0,0 +1,27 @@ + + +# Documentation + +To build the documentation: + +#. Install [Doxygen][doxygen]. Review Comment: nit: I prefer inline link

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

raulcd commented on code in PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#discussion_r1914657246 ## docs/README.md: ## @@ -0,0 +1,27 @@ + + +# Documentation + +To build the documentation: + +#. Install [Doxygen][doxygen]. Review Comment: Was this supposed to be

Re: [PR] Add Doxygen for generating API documentation [iceberg-cpp]

lidavidm commented on PR #27: URL: https://github.com/apache/iceberg-cpp/pull/27#issuecomment-2588799249 How it looks:  -- This is an automated message from the Apache Git Service. To respo