CreateSpark

[timtogan]

This add a createSpark() method. It returns a SparkBase initialized to a SparkMax/Flex depending on the motorConfig passed in.

[Copilot]

Pull request overview

This PR adds a createSpark() factory method that provides a unified interface for creating either SparkMax or SparkFlex motor controllers based on the MotorConfig parameter. This simplifies client code by allowing dynamic selection of the appropriate Spark controller type without needing to explicitly call createSparkMax() or createSparkFlex().

Changes:

• Added two overloaded createSpark() methods that dispatch to the appropriate createSparkMax() or createSparkFlex() method based on the motorConfig.controllerType
• The first overload uses default Spark configuration, while the second accepts a custom SparkBaseConfig

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The default case returns null, which could lead to NullPointerExceptions if an unexpected controllerType is passed. While this pattern is consistent with other methods in this class (e.g., getConfigAccessor at line 157-158, createConfig at line 161-171, getControllerType at line 180), consider whether it would be safer to throw an IllegalArgumentException with a descriptive message explaining that the controllerType is unsupported. This would make debugging easier and fail faster if an invalid configuration is used.

Suggested change

	return null;
	throw new IllegalArgumentException(
	"Unsupported controllerType in MotorControllerFactory.createSpark: "
	+ motorConfig.controllerType);

[timtogan]

@CoolSpy3 could you review pls

[CoolSpy3]

Two small notes.

[timtogan]

@CoolSpy3 Done 👍

[CoolSpy3]

Great; thanks! This looks good ✅ Just fix the javadoc errors. (I think the problem might be that the ...s are causing Javadoc to look for varargs declarations that don't exist, but idk for sure.)

[CoolSpy3]

Nvm. Sorry, GitHub was showing me the not-most-recent commit :P