-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
docs: console.log invaluable typo #9234
Conversation
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.
PR Summary
Minor wording change in frontend documentation to improve clarity around console.log usage guidance.
- Changed "invaluable" to "valuable" in
packages/twenty-website/src/content/developers/frontend-development/best-practices-front.mdx
console.logs section for more straightforward language
💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!
1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile
@@ -149,7 +149,7 @@ They are often not necessary and will make the code harder to read and maintain | |||
|
|||
## Console.logs | |||
|
|||
`console.log` statements are invaluable during development, offering real-time insights into variable values and code flow. But, leaving them in production code can lead to several issues: | |||
`console.log` statements are valuable during development, offering real-time insights into variable values and code flow. But, leaving them in production code can lead to several issues: | |||
|
|||
1. **Performance**: Excessive logging can affect the runtime performance, specially on client-side applications. |
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.
syntax: typo: 'specially' should be 'especially'
Personally, console.logs are invaluable to me :) |
Log
|
Console.logs are valuable in development so changed
invaluable
tovaluable