Skip to content

Stage 1B: Explain what a task is for/will accomplish - #183

Open
TheComputer314 wants to merge 9 commits into
frcsoftware:mainfrom
TheComputer314:FixStage1BComments
Open

Stage 1B: Explain what a task is for/will accomplish#183
TheComputer314 wants to merge 9 commits into
frcsoftware:mainfrom
TheComputer314:FixStage1BComments

Conversation

@TheComputer314

@TheComputer314 TheComputer314 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Description

What changed? Why? (the code + comments should speak for itself on the "how")

Some tasks failed to explain what they are for/what they accomplish. This PR adds extra lines to fix that.

Draft for now as I only did a quick sweep and want other eyes to catch more locations.

Closes #167.

Meta

Merge checklist:

@github-actions github-actions Bot added Curriculum material Curriculum materials, lessons for students, etc stage1 labels Aug 10, 2026
Comment thread src/content/docs/learning-course/stage1/stage1b/command-based-kitbot.mdx Outdated
Comment thread src/content/docs/learning-course/stage1/stage1b/command-based-kitbot.mdx Outdated
@github-actions

Copy link
Copy Markdown

🌐 Preview URL: https://pr-183.frcsoftware.pages.dev

@TheComputer314
TheComputer314 marked this pull request as ready for review August 12, 2026 16:40
4 on Systemcore bus 0.
</TabItem>
</Tabs>
2. A `shoot()` command that waits 2 seconds, then sets the throttle of the motor

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Each bullet point needs to be on a newline. ATM, it seems to render poorly:

Image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixing this fails prettier formatting, prettier for some reason likes that one more.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

For now I fixed it by using explicit HTML tags, though this creates a new issue where uninlined <li> tags create unnecessary and ugly padding between the list elements.
image
TODO: Find a way to make it not do that

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

Labels

Curriculum material Curriculum materials, lessons for students, etc examples stage1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stage 1b: Comments/Explantions

4 participants