-
Notifications
You must be signed in to change notification settings - Fork 1
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
stage 1 #16
Conversation
@@ -8,6 +8,27 @@ import { optimize } from 'svgo'; | |||
|
|||
import type { Plugin, ViteDevServer } from 'vite'; | |||
|
|||
const name = "Alice"; |
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.
The variables name
and age
should be defined in a more meaningful context or passed as parameters to the function instead of being hardcoded.
* | ||
* @returns {{ name: string, age: number }} | ||
*/ | ||
export const doSomething = () => { |
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.
The function name doSomething
is vague and does not convey its purpose. Consider renaming it to something more descriptive.
const name = "Alice"; | ||
const age = 25; | ||
|
||
/** |
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.
The comment block above the function should be removed as it contains a TODO and does not provide immediate value to the code.
@@ -8,6 +8,27 @@ import { optimize } from 'svgo'; | |||
|
|||
import type { Plugin, ViteDevServer } from 'vite'; | |||
|
|||
const name = "Alice"; |
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.
The variables name
and age
should be declared within the function doSomething
to avoid polluting the global scope.
* | ||
* @returns {{ name: string, age: number }} | ||
*/ | ||
export const doSomething = () => { |
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.
The function doSomething
should have a more descriptive name that reflects its intended functionality.
const name = "Alice"; | ||
const age = 25; | ||
|
||
/** |
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.
The comment block above the function should be removed as it does not provide any useful information about the current implementation.
No description provided.