Starlight220

[commands] Revamp Interruptible

3

Still need to think a bit more about how to do this.

Migrate C++ command-based to use `unique_ptr` semantics

2

This is mostly a change in documentation/recommended usage. The current (value) semantics of C++ command-based cause object slicing, which is difficult for both team and WPILib code. For decorators, there's...

[tracking] Command-based improvements

1

This refers to specifically commands themselves, not the button bindings stuff (that's in #4218). All this is subject to change and happily up for discussion. - [x] #4245 - [x]...

[design-discussion] Command interruptibility/runsWhenDisabled rework

2

Due to this being a relatively small change, I'm opening this issue for design discussion as opposed to a full design doc. I'm also bringing together the discussion of `interruptible`...

[cpp commands] Migrate to unique_ptr

2

closes #4303 Once this is more advanced, check whether renaming `CommandPtr` to `Command` (removing the current `Command` class at the head of the inheritance hierarchy) would help reducing breaking changes...

This is all subject to change etc. Up for discussion. - [x] Initial API: #4104 - [x] Add examples - [x] wpilibsuite/frc-docs#1827 - [x] #1888 - at the `BooleanEvent` level...

[command] Rename trigger methods

7

`NetworkButton` should be pulled up to be a direct subclass of `NetworkEvent` (naming TBD); the alternative solution for its use-case--NT(4) listeners--isn't as much of a solution for the same problem-space...

Document AxisType

2

> It's probably this: ```cpp enum AxisType { kXAxis, kYAxis, kZAxis, kTwistAxis, kThrottleAxis }; ``` _Originally posted by @calcmogul in https://github.com/wpilibsuite/allwpilib/pull/4230#issuecomment-1126987665_