Skip to content

fix: fixed misplaced parenthesis #28113

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

Merged
merged 4 commits into from
Jul 27, 2025
Merged

fix: fixed misplaced parenthesis #28113

merged 4 commits into from
Jul 27, 2025

Conversation

dariasavinova
Copy link
Contributor

Description

Adjusted parenthesis placement in the initialization expression to distinguish it from the subsequent for-loop, where they were previously identical.

Motivation

This change clarifies the distinct bracketing structure

Additional details

Related issues and pull requests

Wrapped the entire initialization expression to correct the syntax
@dariasavinova dariasavinova requested a review from a team as a code owner July 24, 2025 08:38
@dariasavinova dariasavinova requested review from lex111 and removed request for a team July 24, 2025 08:38
@github-actions github-actions bot added the l10n-ru Issues related to Russian content. label Jul 24, 2025
@dariasavinova
Copy link
Contributor Author

My change correctly reflects the original intent and matches the initial documentation
However, the linter test fails and suggests reverting to an incorrect (previous) version

Copy link
Contributor

github-actions bot commented Jul 27, 2025

Preview URLs

(comment last updated: 2025-07-27 17:31:41)

Copy link
Member

@saionaro saionaro left a comment

Choose a reason for hiding this comment

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

@dariasavinova привет! Спасибо за правку!

Я отключил линтер для этого примера (как в оригинале)

@saionaro saionaro enabled auto-merge (squash) July 27, 2025 17:29
@saionaro saionaro merged commit 17b1f68 into mdn:main Jul 27, 2025
6 checks passed
@dariasavinova dariasavinova deleted the patch-2 branch July 28, 2025 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ru Issues related to Russian content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants