-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
Ru: translation of simple-profiling.md (II part) #2579
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
Conversation
|
@vasbaz, Maybe you should remove the last empty space for some sentences ;) |
|
@nodejs/nodejs-ru, Please have a review of his translation except for the formation problem, thanks! |
| regular expressions, generating random salts, generating unique hashes from user | ||
| passwords, or inside the Express framework itself. | ||
| По выводу видно, что обрабатывается только около 5 запросов в секунду, а средний | ||
| запрос занимает чуть менее 4 секунд в оба конца. В реальной ситуации могло бы |
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.
не "средний запрос", а "запрос в среднем"
|
@vasbaz can you take care of @alexandrtovmach's comment and fix tests? |
|
Remove the last spaces on the lines See : CI -> Run tests [htmllint] found 0 errors out of 30 files
locale/ru/docs/guides/simple-profiling.md: 126: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
locale/ru/docs/guides/simple-profiling.md: 127: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
locale/ru/docs/guides/simple-profiling.md: 128: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
locale/ru/docs/guides/simple-profiling.md: 129: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
locale/ru/docs/guides/simple-profiling.md: 130: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
locale/ru/docs/guides/simple-profiling.md: 133: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
locale/ru/docs/guides/simple-profiling.md: 134: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
locale/ru/docs/guides/simple-profiling.md: 145: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1] |
|
@XhmikosR @Aissaoui-Ahmed |
|
If you agree with the other changes, you can just apply the fixes and move forward. |
|
I don't any changes here, just requested changes =) |
|
@alexandrtovmach I mean this one
I don't speak Russian, but if you want this change just edit the file :P |
|
@XhmikosR Yeah, but also need to update lint issues... and here just a part of file translated |
| of a user request but even in our simple example, time could be lost compiling | ||
| regular expressions, generating random salts, generating unique hashes from user | ||
| passwords, or inside the Express framework itself. | ||
| По выводу видно, что обрабатывается только около 5 запросов в секунду, а средний |
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.
| По выводу видно, что обрабатывается только около 5 запросов в секунду, а средний | |
| По выводу видно, что обрабатывается только около 5 запросов в секунду, а запрос |
| regular expressions, generating random salts, generating unique hashes from user | ||
| passwords, or inside the Express framework itself. | ||
| По выводу видно, что обрабатывается только около 5 запросов в секунду, а средний | ||
| запрос занимает чуть менее 4 секунд в оба конца. В реальной ситуации могло бы |
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.
| запрос занимает чуть менее 4 секунд в оба конца. В реальной ситуации могло бы | |
| в среднем занимает чуть менее 4 секунд в оба конца. В реальной ситуации могло бы |
| understanding of the performance of your Node.js applications. | ||
|
|
||
| [profiler inside V8]: https://v8.dev/docs/profile | ||
| [профайлер V8]: https://v8.dev/docs/profile |
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.
потому что ссылка не будет работать
| [профайлер V8]: https://v8.dev/docs/profile | |
| [profiler inside V8]: https://v8.dev/docs/profile |
|
That's OK I trimmed the whitespaces. |
|
I'll update file by my own |
No description provided.