Skip to content

Update AV1701: Use US English#370

Open
dennisdoomen wants to merge 1 commit intodevelopfrom
copilot/pr298-update-av1701
Open

Update AV1701: Use US English#370
dennisdoomen wants to merge 1 commit intodevelopfrom
copilot/pr298-update-av1701

Conversation

@dennisdoomen
Copy link
Copy Markdown
Owner

This PR updates guideline AV1701.

It was split out of #298 so the change can be reviewed independently.

Files:

  • _rules/1701.md

Part of the replacement for #298.

Split from #298.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Choose easily readable, preferably grammatically correct names. For example, `HorizontalAlignment` is more readable than `AlignmentHorizontal`.
- Favor readability over brevity. The property name `CanScrollHorizontally` is better than `ScrollableX` (an obscure reference to the X-axis).
- Avoid using names that conflict with keywords of widely used programming languages.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Was this removed intentionally? It would mean declaring a parameter or variable named "event" is fine now (it needs to be escaped, of course: @event). Beginner developers may struggle to pass named parameters like that.

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.

2 participants