You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pr@cassandra.apache.org by GitBox <gi...@apache.org> on 2020/05/06 13:56:14 UTC
[GitHub] [cassandra-website] ossarga commented on a change in pull request #8: Update README.md for new simpler development cycle
ossarga commented on a change in pull request #8:
URL: https://github.com/apache/cassandra-website/pull/8#discussion_r420803485
##########
File path: README.md
##########
@@ -1,10 +1,23 @@
-Apache Cassandra website builder
-================================
+Apache Cassandra website
+========================
+
+Development Cycle
+-----------------
+
+Making changes to the website is done with the following steps.
+
+1. Test changes locally
+2. Commit changes in `src/` to a fork and branch, and create a pull request
+3. Get the pull request reviewed and merged to `master`
+4. Test on https://cassandra.staged.apache.org/ (wait til [ci-cassandra.apache.org](https://ci-cassandra.apache.org/job/cassandra-website/) has deployed it)
Review comment:
Nitpick - reword slightly to say:
> Preview the rendered site on ...
##########
File path: README.md
##########
@@ -1,10 +1,23 @@
-Apache Cassandra website builder
-================================
+Apache Cassandra website
+========================
+
+Development Cycle
+-----------------
+
+Making changes to the website is done with the following steps.
+
+1. Test changes locally
+2. Commit changes in `src/` to a fork and branch, and create a pull request
+3. Get the pull request reviewed and merged to `master`
+4. Test on https://cassandra.staged.apache.org/ (wait til [ci-cassandra.apache.org](https://ci-cassandra.apache.org/job/cassandra-website/) has deployed it)
+5. Merge `asf-staging` to `asf-site`
+6. Test on https://cassandra.apache.org/
Review comment:
Similar to previous comment. May be say something like:
> View the rendered site on ...
##########
File path: README.md
##########
@@ -1,10 +1,23 @@
-Apache Cassandra website builder
-================================
+Apache Cassandra website
+========================
+
+Development Cycle
+-----------------
+
+Making changes to the website is done with the following steps.
+
+1. Test changes locally
+2. Commit changes in `src/` to a fork and branch, and create a pull request
+3. Get the pull request reviewed and merged to `master`
+4. Test on https://cassandra.staged.apache.org/ (wait til [ci-cassandra.apache.org](https://ci-cassandra.apache.org/job/cassandra-website/) has deployed it)
+5. Merge `asf-staging` to `asf-site`
+6. Test on https://cassandra.apache.org/
-Building the Apache Cassandra website takes a number of steps. To make things easier we have provided a Docker container which can build the full website in two simple commands and have it ready to commit via git. If you are interested in the process head over to the [README](./src/README) in _src_ directory.
-Prerequisite
-------------
+To test changes before committing, it is required to build the website locally. Building the Apache Cassandra website takes a number of steps. To make things easier we have provided a Docker container which can build the full website in two simple commands and have it ready to commit via git. If you are interested in the process head over to the [README](./src/README) in _src_ directory.
Review comment:
Might need to reword the first sentence. Specifically add a few words to the part after the comma. Feel free to use any of the following variations:
> To test changes before committing, it is a requirement that you build the website locally.
OR
> To test changes before committing, it is required that you build the website locally.
OR (completely reword it)
> To test changes before committing, you will need to build the website locally.
----------------------------------------------------------------
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:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org