Copilot commented on code in PR #7437:
URL: https://github.com/apache/ignite-3/pull/7437#discussion_r2706265848


##########
modules/platforms/cpp/Doxyfile:
##########
@@ -2906,7 +2906,9 @@ INPUT= \
     ignite/common \
     docs/index.dox
 
-;STRIP_FROM_PATH=
+PROJECT_NUMBER = $(IGNITE_VERISON)

Review Comment:
   The environment variable name is misspelled. It should be "IGNITE_VERSION" 
instead of "IGNITE_VERISON" (missing 'I' before 'ON'). This needs to match the 
environment variable set in build.gradle.
   ```suggestion
   PROJECT_NUMBER = $(IGNITE_VERSION)
   ```



##########
modules/platforms/build.gradle:
##########
@@ -81,6 +81,7 @@ def aggregateDotnetDocs = 
tasks.register('aggregateDotnetDocs', Copy) {
 
 def doxygenCppClient = tasks.register('doxygenCppClient', Exec) {
     workingDir "$projectDir/cpp"
+    environment("IGNITE_VERISON", "${project.version}")

Review Comment:
   The environment variable name is misspelled. It should be "IGNITE_VERSION" 
instead of "IGNITE_VERISON" (missing 'I' before 'ON'). This needs to match the 
variable referenced in Doxyfile.
   ```suggestion
       environment("IGNITE_VERSION", "${project.version}")
   ```



##########
modules/platforms/cpp/Doxyfile:
##########
@@ -48,7 +48,7 @@ PROJECT_NAME           = "Apache Ignite C++ Client"
 # could be handy for archiving the generated documentation or if some version
 # control system is used.
 
-PROJECT_NUMBER         =
+#PROJECT_NUMBER         =

Review Comment:
   The original PROJECT_NUMBER definition at line 51 is commented out but a new 
one is defined at line 2909. Consider removing the commented line entirely to 
avoid confusion, as it no longer serves any purpose.
   ```suggestion
   
   ```



-- 
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]

Reply via email to