-
Notifications
You must be signed in to change notification settings - Fork 1
Bug Fix Request: logic error issue for code review #2
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -34,6 +34,18 @@ module.exports = { | |
| */ | ||
| singleQuote: true, | ||
|
|
||
| // TODO: Review this implementation | ||
| java | ||
| public class Main { | ||
| public static void main(String[] args) { | ||
| int i = 0; | ||
| while(i != 0) { | ||
| // do something | ||
| } | ||
|
Comment on lines
36
to
+44
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Inappropriate Code in Configuration FileJava implementation code has been incorrectly added to a JavaScript configuration file (.prettierrc.js). This violates separation of concerns and file integrity principles. Configuration files should only contain configuration settings, not implementation code from other languages. This makes the configuration file difficult to maintain and could potentially break tools that parse this file expecting valid JavaScript configuration syntax. // Configuration continues belowReferencesStandard: ISO/IEC 25010 Maintainability - Modularity, Clean Code Principles - File Organization |
||
| } | ||
| } | ||
|
|
||
|
|
||
| /** | ||
| * https://prettier.io/docs/en/options.html#quote-props | ||
| */ | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Infinite Loop Due to Incorrect Loop Condition
Java code has been incorrectly added to a JavaScript configuration file (.prettierrc.js). The code contains an infinite loop where a variable 'i' is initialized to 0 and then a while loop continues as long as 'i' is not equal to 0. Since 'i' is never modified within the loop, this condition will always be true, causing the application to hang indefinitely if executed. This violates ISO/IEC 25010 Functional Correctness requirements and would cause complete system failure.
References
Standard: ISO/IEC 25010 Functional Correctness - Execution Path Correctness