Conversation
|
ESLint detected 22 issues on 132963b posted by Sider |
postinstall command install all the electron app native deps.
|
ESLint analysis of 66aebbb failed. posted by Sider |
|
stylelint analysis of 66aebbb failed. posted by Sider |
src/renderer/mixins/addRepository.js
Outdated
| FROM repository | ||
| INNER JOIN gitRepository USING(repositoryId)`, | ||
| (err, data) => { | ||
| if (err) console.log(err); |
There was a problem hiding this comment.
[ESLint] Warning - Unexpected console statement. (no-console) (link)
posted by Sider
| (err, data) => { | ||
| if (err) console.log(err); | ||
| else { | ||
| console.log(data); |
There was a problem hiding this comment.
[ESLint] Warning - Unexpected console statement. (no-console) (link)
posted by Sider
| ); | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
[ESLint] Warning - Insert ; (prettier/prettier) (link)
posted by Sider
| ); | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
[ESLint] Warning - Missing semicolon. (semi) (link)
posted by Sider
| FROM repository | ||
| INNER JOIN gitRepository USING(repositoryId)`, | ||
| (err, data) => { | ||
| if (err) console.log(err); |
There was a problem hiding this comment.
[ESLint] Warning - Unexpected console statement. (no-console) (link)
posted by Sider
Update and simplify database columns
| $repositoryId: data.repositoryId | ||
| }, | ||
| (err, data) => { | ||
| if (err) console.log("err", err); |
There was a problem hiding this comment.
[ESLint] Warning - Unexpected console statement. (no-console) (link)
posted by Sider
Switch from vuex to sqlite database.
| $commandId: commandId | ||
| }, | ||
| err => { | ||
| if (err) console.log(err); |
There was a problem hiding this comment.
[ESLint] Warning - Unexpected console statement. (no-console) (link)
posted by Sider
| $path: path | ||
| }, | ||
| err => { | ||
| if (err) console.log(err); |
There was a problem hiding this comment.
[ESLint] Warning - Unexpected console statement. (no-console) (link)
posted by Sider
|
Closing this PR against #3577 |
Currently, we are using vuex to store data inside the states.