self review

src/core/server/logging/appenders/rolling_file/rolling_file_appender.ts

@@ -166,7 +166,7 @@ export class RollingFileAppender implements DisposableAppender {
   }
 
   /**
-   * Checks if the current even should trigger a rollout
+   * Checks if the current event should trigger a rollout
    */
   private needRollout(record: LogRecord) {
     return this.policy.isTriggeringEvent(record);

src/core/server/logging/appenders/rolling_file/strategies/numeric/pattern_matcher.ts

@@ -42,7 +42,7 @@ const createNumericMatcher = (fileBaseName: string, pattern: string): RegExp =>
  * matcher('kibana-1.log') // `1`
  * matcher('kibana-5.log') // `5`
  * matcher('kibana-A.log') // undefined
- * matcher('kibana.log')   // `undefined
+ * matcher('kibana.log')   // undefined
  * ```
  */
 export const getFileNameMatcher = (logFileName: string, pattern: string) => {

src/core/server/logging/appenders/rolling_file/strategies/numeric/rolling_tasks.ts

@@ -28,6 +28,9 @@ export const shouldSkipRollout = async ({ logFilePath }: { logFilePath: string }
   return !(await exists(logFilePath));
 };
 
+/**
+ * Returns the rolled file basenames, from the most recent to the oldest.
+ */
 export const getOrderedRolledFiles = async ({
   logFileBaseName,
   logFileFolder,

src/core/server/logging/appenders/rolling_file/strategies/strategy.ts

@@ -18,7 +18,7 @@
  */
 
 /**
- * A strategy to perform the log file rollout.
+ * A strategy to perform the log file rollover.
  */
 export interface RollingStrategy {
   /**