[commands] Add a warning to schedule javadocs

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

@@ -180,6 +180,8 @@ 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.
    *
+   *  WARNING: using this function directly is often a footgun and should be avoided. Instead Triggers should be used to schedule Commands.
+   *
    * @param command the command to schedule. If null, no-op.
    */
   private void schedule(Command command) {
@@ -230,6 +232,8 @@ private void schedule(Command command) {
   /**
    * Schedules multiple commands for execution. Does nothing for commands already scheduled.
    *
+   *  WARNING: using this function directly is often a footgun and should be avoided. Instead Triggers should be used to schedule Commands.
+   *
    * @param commands the commands to schedule. No-op on null.
    */
   public void schedule(Command... commands) {

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

@@ -24,7 +24,6 @@ namespace frc2 {
 class Command;
 class CommandPtr;
 class Subsystem;
-
 /**
  * The scheduler responsible for running Commands.  A Command-based robot should
  * call Run() on the singleton instance in its periodic block in order to run
@@ -88,6 +87,8 @@ 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 avoided. Instead Triggers should be used to schedule Commands.
+   *
    * @param command the command to schedule
    */
   void Schedule(const CommandPtr& command);
@@ -99,6 +100,8 @@ class CommandScheduler final : public wpi::Sendable,
    * scheduled as 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 avoided. Instead Triggers should be used to schedule Commands.
+   *
    * @param command the command to schedule
    */
   void Schedule(Command* command);
@@ -107,6 +110,8 @@ 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 avoided. Instead Triggers should be used to schedule Commands.
+   *
    * @param commands the commands to schedule
    */
   void Schedule(std::span<Command* const> commands);
@@ -115,6 +120,8 @@ 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 avoided. Instead Triggers should be used to schedule Commands.
+   *
    * @param commands the commands to schedule
    */
   void Schedule(std::initializer_list<Command*> commands);