minor doc changes

Author: acdvs

docs/rules/module-vars.md

@@ -2,16 +2,9 @@
 
 Enforces correct module identifiers for each configured module name.
 
-## Rule Details
-
 Requires at least one module name to be specified to take effect.
 
-```js
-'suitescript/module-vars': [<enabled>, {
-  moduleName: <string>
-  ...
-}]
-```
+## Rule Details
 
 :white_check_mark: The following patterns are **correct**:
 
@@ -39,6 +32,17 @@ define(['N/record'], function(rec) {});
 define(['N/ui/serverWidget'], function(ui) {});
 ```
 
+## Rule Options
+
+Specify at least one module name with a corresponding variable name.
+
+```js
+'suitescript/module-vars': [<enabled>, {
+  moduleName: <string>
+  ...
+}]
+```
+
 ## Module Names
 
 - N/action

docs/rules/no-amd-name.md

@@ -1,7 +1,11 @@
 # suitescript/no-amd-name
 
+:wrench: _The `--fix` option on the command line can automatically fix some of the problems reported by this rule._
+
 Restricts naming of AMD modules.
 
+Naming AMD modules [should generally be avoided](https://requirejs.org/docs/whyamd.html#namedmodules) because compilers will handle the ID-ing of modules internally where necessary.
+
 ## Rule Details
 
 :white_check_mark: The following pattern is **correct**:

docs/rules/no-extra-modules.md

@@ -1,6 +1,6 @@
 # suitescript/no-extra-modules
 
-Enforces an equal number of module literals and identifiers in the `define` call.
+Enforces an equal number of module literals and identifiers in the `define` call arguments.
 
 ## Rule Details
 

docs/rules/no-invalid-modules.md

@@ -1,6 +1,6 @@
 # suitescript/no-invalid-modules
 
-Enforces valid SuiteScript module names in `define` array.
+Enforces valid SuiteScript module names in the `define` array.
 
 ## Rule Details
 

docs/rules/no-log-module.md

@@ -2,10 +2,12 @@
 
 Restricts loading of the N/log module in favor of global `log`.
 
-> **Note:** By default, the N/log module is allowed in client scripts because there it is used to log to NetSuite where it would otherwise be impossible. See the rule options below to modify this.
+In most cases, global `log` can be used instead of the N/log module.
 
 ## Rule Details
 
+By default, the N/log module is allowed in client scripts because there it is used to log to NetSuite where it would otherwise be impossible. See the rule options below to modify this.
+
 :white_check_mark: The following pattern is **correct**:
 
 ```js

docs/rules/no-module-extensions.md

@@ -1,5 +1,7 @@
 # suitescript/no-module-extensions
 
+:wrench: _The `--fix` option on the command line can automatically fix some of the problems reported by this rule._
+
 Enforces no filename extensions on module dependencies.
 
 > **Note:** As of January 2021, there appears to be a NetSuite bug that intermittently causes an error (`SuiteScript 2.1 entry point scripts must implement one script type function..`) when attempting to upload script files that include dependencies with file extensions.