Skip to content

Add files via upload - #7

Open
GAS53 wants to merge 1 commit into
mainfrom
lesson_6
Open

Add files via upload#7
GAS53 wants to merge 1 commit into
mainfrom
lesson_6

Conversation

@GAS53

@GAS53 GAS53 commented Apr 17, 2022

Copy link
Copy Markdown
Owner

No description provided.

@Akseonov Akseonov left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

все хорошо, замечаний нет

Comment thread app.js
Comment on lines +14 to +15
const title = colors[randomInt(0, colors.length - 1)] + ' ' + entities[randomInt(0, entities.length - 1)];
return title

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

переменная title избыточна, можно было сразу вернуть все то, что присваивается title

Comment thread app.js
Comment on lines +85 to +86
let = productRow = `
<tr>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

после let знак равно лишний

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants