Skip to content

Shooter Subsystem 初稿#6

Open
ttjanet wants to merge 3 commits intomainfrom
shooter
Open

Shooter Subsystem 初稿#6
ttjanet wants to merge 3 commits intomainfrom
shooter

Conversation

@ttjanet
Copy link

@ttjanet ttjanet commented Jan 29, 2026

shooter的初版程式

public ShooterSubsystem() {}

private void shoot() {
private final PWMSparkMax shooter = new PWMSparkMax(0);
Copy link
Contributor

Choose a reason for hiding this comment

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

我們用的是 CAN 訊號的 sparkMAX 喔
所以應該是 SparkMax 這個類別

@Hannahjjj97
Copy link
Contributor

阿你如果改完了 要再 request review 一次喔

Copy link
Contributor

@Hannahjjj97 Hannahjjj97 left a comment

Choose a reason for hiding this comment

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

你的 commit message 沒有照規定喔
需修正

{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
Copy link
Contributor

Choose a reason for hiding this comment

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

這個檔案不應該修改到喔
請改回來

@Hannahjjj97
Copy link
Contributor

你的 commit message 沒有照規定喔
需修正

commit message 如果很難修正就算了,但下次記得照規定

public ShooterSubsystem() {}

private void shoot() {
private final SparkMax shooter = new SparkMax(0, null);

Choose a reason for hiding this comment

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

忘記給 type 了吧?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants