Skip to content

Conversation

@ThadHouse
Copy link
Member

I forgot to add this.

@ThadHouse ThadHouse requested a review from a team as a code owner October 29, 2025 02:58
@github-actions github-actions bot added the 2027 2027 target label Oct 29, 2025
Copy link
Member

@SamCarlberg SamCarlberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just needs a constructor overload

@SamCarlberg SamCarlberg added the component: commands v3 WPILib commands library version 3 label Oct 29, 2025
Comment on lines +421 to +432
public Trigger misc1() {
return misc1(getScheduler().getDefaultEventLoop());
}

/**
* Constructs a Trigger instance around the Miscellaneous 1 button's digital signal.
*
* @param loop the event loop instance to attach the event to.
* @return a Trigger instance representing the Miscellaneous 1 button's digital signal attached to
* the given loop.
*/
public Trigger misc1(EventLoop loop) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't these be near the other miscellaneous buttons?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the order they're defined in in SDL, and was the way it was generated.

…mepad.java

Co-authored-by: Dan Katzuv <31829093+katzuv@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2027 2027 target component: commands v3 WPILib commands library version 3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants