docs/spec/v2beta2: address review comments

Signed-off-by: Hidde Beydals <[email protected]>

docs/spec/v2beta2/helmreleases.md

@@ -2,8 +2,8 @@
 
 The `HelmRelease` API allows for controller-driven reconciliation of Helm
 releases via Helm actions such as install, upgrade, test, uninstall, and
-rollback. In addition to this, it allows detecting and correcting cluster state
-drift from the desired release state.
+rollback. In addition to this, it detects and corrects cluster state drift
+from the desired release state.
 
 ## Example
 
@@ -68,9 +68,9 @@ In the above example:
  Helm release exists and is up-to-date.
 - If the Helm release does not exist, is not up-to-date, or has not observed to
  be made by the controller, then the controller will install or upgrade the
- release, respectively. If this fails, it is allowed to retry the operation
- a number of times while requeueing between attempts, as defined by the 
- respective [remediation configurations](#configuring-failure-handling).
+ release. If this fails, it is allowed to retry the operation a number of
+ times while requeueing between attempts, as defined by the respective
+ [remediation configurations](#configuring-failure-handling).
 - If the [Helm tests](#test-configuration) for the release have not been run
  before for this release, the HelmRelease will run them.
 - When the Helm release in storage is up-to-date, the controller will check if
@@ -451,7 +451,7 @@ The field offers the following subfields:
  Defaults to `false`.
 - `.preserveValues` (Optional): Instructs Helm to re-use the values from the
  last release while merging in overrides from [values](#values). Setting
- this flag makes the HelmRelease non-declarative.
+ this flag makes the HelmRelease non-declarative. Defaults to `false`.
 
 #### Upgrade remediation
 
@@ -552,8 +552,10 @@ The field offers the following subfields:
 - `.timeout` (Optional): The time to wait for any individual Kubernetes
  operation (like Jobs for hooks) during the uninstalltion of the release.
  Defaults to the [global timeout value](#timeout).
-- `.deletionPropagation` (Optional):
- `.disableHooks` (Optional): Prevents [chart hooks](https://helm.sh/docs/topics/charts_hooks/)
+- `.deletionPropagation` (Optional): The [deletion propagation policy](https://kubernetes.io/docs/tasks/administer-cluster/use-cascading-deletion/)
+ when a Helm uninstall is performed. Valid values are `background`,
+ `foreground` and `orphan`. Defaults to `background`.
+- `.disableHooks` (Optional): Prevents [chart hooks](https://helm.sh/docs/topics/charts_hooks/)
  from running during the uninstallation of the release. Defaults to `false`.
 - `.disableWait` (Optional): Disables waiting for resources to be deleted after
  uninstalling the release. Defaults to `false`.
@@ -569,7 +571,7 @@ storage.
 
 When `.spec.driftDetection.mode` is set to `warn` or `enabled`, and the
 desired state of the HelmRelease is in-sync with the Helm release object in
-the storage. The controller will compare the manifest from the Helm storage
+the storage, the controller will compare the manifest from the Helm storage
 with the current state of the cluster using a
 [server-side dry-run apply](https://kubernetes.io/docs/reference/using-api/server-side-apply/).
 
@@ -581,9 +583,9 @@ to the controller logs (with `--log-level=debug`).
 
 #### Drift correction
 
-When `.spec.driftDetection.mode` is set to `enabled`, the controller will
-attempt to correct the drift by creating and patching the resources based on
-the server-side dry-run result.
+Furthermore, when `.spec.driftDetection.mode` is set to `enabled`, the
+controller will attempt to correct the drift by creating and patching the
+resources based on the server-side dry-run result.
 
 At the end of the correction attempt, it will emit a Kubernetes Event with a
 summary of the changes it made and any failures it encountered. In case of a