-
Notifications
You must be signed in to change notification settings - Fork 610
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[cmd] Add a warning to schedule
javadocs
#7073
base: main
Are you sure you want to change the base?
Conversation
This PR modifies commands. Please open a corresponding PR in Python Commands and include a link to this PR. |
The preferred design pattern should be explained, not just a warning about use. |
7cf1e9c
to
b427b2e
Compare
/format |
schedule
javadocsschedule
javadocs
I think you can use @warning for the c++ |
d003a37
to
6eb9b24
Compare
/format |
6eb9b24
to
32a9c22
Compare
/format |
@sciencewhiz can you review? |
16c19f5
to
2a1178c
Compare
/format |
wpilibNewCommands/src/main/native/include/frc2/command/CommandScheduler.h
Outdated
Show resolved
Hide resolved
wpilibNewCommands/src/main/native/include/frc2/command/CommandScheduler.h
Show resolved
Hide resolved
46f9b37
to
2613107
Compare
Signed-off-by: Jade Turner <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Footgun is jargon. Can we say something like “can lead to unexpected behavior”?
Signed-off-by: Jade Turner <[email protected]>
/format |
wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/CommandScheduler.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Jade Turner <[email protected]>
87b450a
to
4d5cc46
Compare
/format |
No description provided.