Skip to content

Stage 0: fixed variable names in Conditionals - #137

Merged
Adrianamm merged 7 commits into
frcsoftware:mainfrom
Adrianamm:conditionals-mistake
Jul 27, 2026
Merged

Stage 0: fixed variable names in Conditionals#137
Adrianamm merged 7 commits into
frcsoftware:mainfrom
Adrianamm:conditionals-mistake

Conversation

@Adrianamm

Copy link
Copy Markdown
Member

Description

What changed? Why? (the code + comments should speak for itself on the "how")
Changed the variable names in the text, code, and flowchart so it follows camelCase

Closes #118

Meta

Merge checklist:

@Adrianamm
Adrianamm requested a review from a team as a code owner July 26, 2026 16:31
@github-actions github-actions Bot added Curriculum material Curriculum materials, lessons for students, etc examples infra Any infrastructure for building the website or syncing files stage0 labels Jul 26, 2026
@Adrianamm Adrianamm changed the title fixed variable names Stage O: fixed variable names in Conditionals Jul 26, 2026
@Adrianamm Adrianamm changed the title Stage O: fixed variable names in Conditionals Stage 0: fixed variable names in Conditionals Jul 26, 2026
@github-actions

Copy link
Copy Markdown

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

@Adrianamm
Adrianamm requested a review from zachwaffle4 July 26, 2026 16:59

@reniejoshi reniejoshi left a comment

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.

LGTM, make sure merge main into this branch

Comment thread .github/ISSUE_TEMPLATE/feature_request.md
0verkil and others added 2 commits July 27, 2026 13:42
## Description

Modified the installation guide in README.md to ensure all prerequisites
were covered and to give correct version requirements for `Node.js`.

Before:
<img width="524" height="164" alt="Screenshot 2026-07-26 at 09 38 46"
src="https://github.com/user-attachments/assets/c8587579-5b35-44c0-b125-950af3ad999b"
/>

After:
<img width="818" height="234" alt="Screenshot 2026-07-26 at 09 39 11"
src="https://github.com/user-attachments/assets/2a1e6cc5-6d7f-4588-8e20-fcda89d30bd9"
/>

## Meta

Merge checklist:

- [X] Pull Request title is [short, imperative
summary](https://cbea.ms/git-commit/) of proposed changes
- [X] The description documents the _what_ and _why_, including events
that led to this PR
- [X] Pull Request has been [linted and
formatted](https://frcsoftware.org/contribution/styleguide/#linting-and-formatting)
and follows the
[styleguide](https://frcsoftware.org/contribution/styleguide)
@github-actions github-actions Bot removed the infra Any infrastructure for building the website or syncing files label Jul 27, 2026
@Adrianamm
Adrianamm merged commit 6cd8737 into frcsoftware:main Jul 27, 2026
7 checks passed
@Adrianamm
Adrianamm deleted the conditionals-mistake branch July 27, 2026 20:22
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 stage0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Variable name consistency in Conditionals.java

6 participants