tobous commented on a change in pull request #14:
URL: https://github.com/apache/db-jdo-site/pull/14#discussion_r606792654



##########
File path: README.md
##########
@@ -7,18 +7,21 @@ The website is mirrored on 
https://apache.github.io/db-jdo-site/.
 
 ## Building the Site
 
-The content and styling of the site is defined in the 
[AsciiDoc](https://asciidoc.org/) format. It is built using 
[Maven](https://maven.apache.org/).
+The content and styling of the site is defined in the 
[AsciiDoc](https://asciidoc.org/) format. It is built using 
[Maven](https://maven.apache.org/). For details on publishing the site see 
section [Publishing the Site](#publishing-the-site).
 
 The site can be built by calling `mvn clean compile`. This generates the HTML 
files in `target/site`.
+Most of the site will work with the exception of the javadoc file downloads.
+If needed, call `mvn package`. This copies the javadoc files to `target/site`.
+The site can then be viewed by opening the local file `target/site/index.html` 
in a browser.
 
 ### Adding Javadoc
 
 The site contains a packaged version of the JDO API javadoc. It can be updated 
as follows:
 
 * Create the javadoc jar (e.g. jdo-api-3.2-javadoc.jar) in the db-jdo 
repository by calling `mvn clean install -Papache-release` in the api submodule.
-* Create a new folder under docs e.g. docs/api32.
-* Copy the javadocs jar info the new folder: e.g. `cp  jdo-api-3.2-javadoc.jar 
 docs/api32`.
-* Create a new subfolder docs/api32/jdo-api-3.2-javadoc
+* Create a new folder under docs e.g. `src/main/resources/javadoc/api32`.

Review comment:
       ```suggestion
   * Create a new folder under in the javadoc resources directory, e.g. 
`src/main/resources/javadoc/api32`.
   ```

##########
File path: README.md
##########
@@ -32,15 +35,27 @@ This repository contains the JDO website source.
 
  * The AsciiDoc sources can be found in `src/main/asciidoc`.
  * The website menu is defined in `src/main/template`.
- * The converter for migrating the old HTML files to AsciiDoc can be found in 
`src/main/java`
+ * The converter for migrating the old HTML files to AsciiDoc can be found in 
`src/main/java`.
+ * Additional pre-compiled resources are located in `src/main/resources`.
 
-Contributions to this repository follow the default [GitHub 
workflow](https://guides.github.com/introduction/flow/) using 
[forks](https://guides.github.com/activities/forking/).
+Contributions to this repository follow the default [GitHub 
workflow](https://guides.github.com/introduction/flow/)
+using [forks](https://guides.github.com/activities/forking/).
 
 To contribute changes, you can follow these steps:
 
  * Adapt the AsciiDoc files in `src/main/asciidoc` or the website menu in  
`src/main/template`.
- * Call `mvn clean compile` to build the site and verify the generated website 
by viewing it locally with a web browser.
- * Commit the source changes (not the build artifacts) and open a pull request.
+ * Build the site (see [above](#building-the-site) and verify the generated 
website by viewing `target/site/index.html` locally with a web browser.

Review comment:
       ```suggestion
    * Build the site (see [above](#building-the-site)) and verify the generated 
website by viewing `target/site/index.html` locally with a web browser.
   ```




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

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


Reply via email to