review

Signed-off-by: Jade Turner <[email protected]>

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/CommandScheduler.java

@@ -180,7 +180,7 @@ private void initCommand(Command command, Set<Subsystem> requirements) {
    * using those requirements have been scheduled as interruptible. If this is the case, they will
    * be interrupted and the command will be scheduled.
    *
-   * <p>WARNING: using this function directly is often a footgun and should be avoided. Instead
+   * <p>WARNING: using this function directly is often can lead to unexpected behavior and should be avoided. Instead
    * Triggers should be used to schedule Commands.
    *
    * @param command the command to schedule. If null, no-op.
@@ -233,7 +233,7 @@ private void schedule(Command command) {
   /**
    * Schedules multiple commands for execution. Does nothing for commands already scheduled.
    *
-   * <p>WARNING: using this function directly is often a footgun and should be avoided. Instead
+   * <p>WARNING: using this function directly is often can lead to unexpected behavior and should be avoided. Instead
    * Triggers should be used to schedule Commands.
    *
    * @param commands the commands to schedule. No-op on null.

wpilibNewCommands/src/main/native/include/frc2/command/CommandScheduler.h

@@ -88,7 +88,7 @@ class CommandScheduler final : public wpi::Sendable,
    * interruptible. If this is the case, they will be interrupted and the
    * command will be scheduled.
    *
-   * @warning Using this function directly is often a footgun and should be
+   * @warning Using this function directly is often can lead to unexpected behavior and should be
    * avoided. Instead Triggers should be used to schedule Commands.
    *
    * @param command the command to schedule
@@ -115,7 +115,7 @@ class CommandScheduler final : public wpi::Sendable,
    *
    * The pointer must remain valid through the entire lifecycle of the command.
    *
-   * @warning Using this function directly is often a footgun and should be
+   * @warning Using this function directly is often can lead to unexpected behavior and should be
    * avoided. Instead Triggers should be used to schedule Commands.
    *
    * @param command the command to schedule
@@ -126,7 +126,7 @@ class CommandScheduler final : public wpi::Sendable,
    * Schedules multiple commands for execution. Does nothing for commands
    * already scheduled.
    *
-   * @warning Using this function directly is often a footgun and should be
+   * @warning Using this function directly is often can lead to unexpected behavior and should be
    * avoided. Instead Triggers should be used to schedule Commands.
    *
    * @param commands the commands to schedule
@@ -137,7 +137,7 @@ class CommandScheduler final : public wpi::Sendable,
    * Schedules multiple commands for execution. Does nothing for commands
    * already scheduled.
    *
-   * @warning Using this function directly is often a footgun and should be
+   * @warning Using this function directly is often can lead to unexpected behavior and should be
    * avoided. Instead Triggers should be used to schedule Commands.
    *
    * @param commands the commands to schedule