Skip to content

버전업 1.9.51 #717

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

Merged
merged 33 commits into from
Feb 22, 2024
Merged

버전업 1.9.51 #717

merged 33 commits into from
Feb 22, 2024

Conversation

Tnks2U
Copy link
Contributor

@Tnks2U Tnks2U commented Feb 22, 2024

irarykim and others added 30 commits January 19, 2024 08:22
fix: robokitRS 펌웨어 수정
HexaBoard_윈도우 연결문제 해결
로보타미 펌웨어 변경 및 시리얼 포드 선택 변경
지니로봇-토이봇 하드웨어 수정
@Tnks2U Tnks2U requested review from kimorkim and prisml February 22, 2024 10:37
@Tnks2U Tnks2U self-assigned this Feb 22, 2024
@Tnks2U Tnks2U merged commit 6425cfa into master Feb 22, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

eslint

app/modules/ITPLE.js|173 col 25| Unexpected function expression.
app/modules/ITPLE.js|173 col 33| Unexpected space before function parentheses.
app/modules/ITPLE.js|174 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|199 col 41| Unexpected space before function parentheses.
app/modules/ITPLE.js|200 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|201 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|209 col 9| Unexpected var, use let or const instead.
app/modules/ITPLE.js|210 col 9| Unexpected var, use let or const instead.
app/modules/ITPLE.js|211 col 51| Unexpected function expression.
app/modules/ITPLE.js|211 col 59| Unexpected space before function parentheses.
app/modules/ITPLE.js|212 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|241 col 45| Unexpected space before function parentheses.
app/modules/ITPLE.js|242 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|246 col 49| Unexpected function expression.
app/modules/ITPLE.js|246 col 57| Unexpected space before function parentheses.
app/modules/ITPLE.js|248 col 31| Unexpected function expression.
app/modules/ITPLE.js|248 col 39| Unexpected space before function parentheses.
app/modules/ITPLE.js|261 col 44| Unexpected space before function parentheses.
app/modules/ITPLE.js|262 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|263 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|265 col 19| Unexpected function expression.
app/modules/ITPLE.js|265 col 27| Unexpected space before function parentheses.
app/modules/ITPLE.js|269 col 9| Unexpected var, use let or const instead.
app/modules/ITPLE.js|270 col 9| Unexpected var, use let or const instead.
app/modules/ITPLE.js|287 col 9| Unexpected var, use let or const instead.
app/modules/ITPLE.js|288 col 9| Unexpected var, use let or const instead.
app/modules/ITPLE.js|338 col 49| Unexpected space before function parentheses.
app/modules/ITPLE.js|339 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|340 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|366 col 9| 'value' is not defined.
app/modules/ITPLE.js|367 col 9| 'value' is not defined.
app/modules/ITPLE.js|369 col 41| 'value' is not defined.
app/modules/ITPLE.js|380 col 45| Unexpected space before function parentheses.
app/modules/ITPLE.js|381 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|382 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|383 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|395 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|429 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|430 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|431 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|432 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|477 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|478 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|487 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|488 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|489 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|490 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|491 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|492 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|493 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|494 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|495 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|496 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|497 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|498 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|499 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|500 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|501 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|502 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|576 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|577 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|578 col 13| Unexpected var, use let or const instead.
app/modules/ITPLE.js|600 col 44| Unexpected space before function parentheses.
app/modules/ITPLE.js|601 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|602 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|603 col 20| Unexpected function expression.
app/modules/ITPLE.js|603 col 28| Unexpected space before function parentheses.
app/modules/ITPLE.js|613 col 39| Unexpected space before function parentheses.
app/modules/ITPLE.js|614 col 5| Unexpected var, use let or const instead.
app/modules/ITPLE.js|621 col 34| Unexpected space before function parentheses.
app/modules/coco.js|164 col 33| Unexpected space before function parentheses.
app/modules/coco.js|166 col 42| Unexpected space before function parentheses.
app/modules/coco.js|171 col 47| Unexpected space before function parentheses.
app/modules/coco.js|177 col 45| Unexpected space before function parentheses.
app/modules/coco.js|186 col 41| Unexpected space before function parentheses.
app/modules/coco.js|193 col 46| Unexpected space before function parentheses.
app/modules/coco.js|197 col 46| Unexpected space before function parentheses.
app/modules/coco.js|209 col 45| Unexpected space before function parentheses.
app/modules/coco.js|281 col 41| Unexpected space before function parentheses.
app/modules/coco.js|325 col 45| Unexpected space before function parentheses.
app/modules/coco.js|345 col 44| Unexpected space before function parentheses.
app/modules/coco.js|456 col 49| Unexpected space before function parentheses.
app/modules/coco.js|510 col 13| 'value' is never reassigned. Use 'const' instead.
app/modules/coco.js|524 col 45| Unexpected space before function parentheses.
app/modules/coco.js|822 col 17| 'textLenBuf' is never reassigned. Use 'const' instead.
app/modules/coco.js|856 col 17| 'list' is never reassigned. Use 'const' instead.
app/modules/coco.js|857 col 17| 'line' is never reassigned. Use 'const' instead.
app/modules/coco.js|858 col 17| 'col' is never reassigned. Use 'const' instead.
app/modules/coco.js|876 col 13| Unexpected var, use let or const instead.
app/modules/coco.js|877 col 13| Unexpected var, use let or const instead.
app/modules/coco.js|878 col 13| Unexpected var, use let or const instead.
app/modules/coco.js|879 col 13| Unexpected var, use let or const instead.
app/modules/coco.js|880 col 13| Unexpected var, use let or const instead.
app/modules/coco.js|880 col 30| A function with a name starting with an uppercase letter should only be used as a constructor.
app/modules/coco.js|883 col 28| Unexpected string concatenation.
app/modules/coco.js|885 col 36| Unexpected string concatenation.
app/modules/coco.js|947 col 19| Identifier 'time_value' is not in camel case.
app/modules/coco.js|961 col 54| Identifier 'time_value' is not in camel case.
app/modules/coco.js|1027 col 44| Unexpected space before function parentheses.
app/modules/coco.js|1040 col 39| Unexpected space before function parentheses.
app/modules/coco.js|1048 col 34| Unexpected space before function parentheses.
app/modules/coco.js|1055 col 43| Unexpected space before function parentheses.
app/modules/jinirobot_toybot.js|14 col 32| Missing trailing comma.
app/modules/jinirobot_toybot.js|17 col 9| A space is required after '{'.
app/modules/jinirobot_toybot.js|17 col 35| A space is required before '}'.
app/modules/jinirobot_toybot.js|18 col 9| A space is required after '{'.
app/modules/jinirobot_toybot.js|18 col 35| A space is required before '}'.
app/modules/jinirobot_toybot.js|19 col 9| A space is required after '{'.
app/modules/jinirobot_toybot.js|19 col 35| A space is required before '}'.
app/modules/jinirobot_toybot.js|20 col 9| A space is required after '{'.
app/modules/jinirobot_toybot.js|20 col 35| A space is required before '}'.
app/modules/jinirobot_toybot.js|21 col 9| A space is required after '{'.
app/modules/jinirobot_toybot.js|21 col 35| A space is required before '}'.
app/modules/jinirobot_toybot.js|21 col 36| Missing trailing comma.
app/modules/jinirobot_toybot.js|22 col 6| Missing semicolon.
app/modules/jinirobot_toybot.js|49 col 9| A space is required after '{'.
app/modules/jinirobot_toybot.js|49 col 35| A space is required before '}'.
app/modules/jinirobot_toybot.js|49 col 36| Missing trailing comma.
app/modules/jinirobot_toybot.js|252 col 2| Missing semicolon.

this.isDraing = false;
}

var sensorIdx = 0;

Choose a reason for hiding this comment

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

⚠️ [eslint] <no-var> reported by reviewdog 🐶
Unexpected var, use let or const instead.

Suggested change
var sensorIdx = 0;
let sensorIdx = 0;


var sensorIdx = 0;

Module.prototype.init = function (handler, config) {};

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.init = function (handler, config) {};
Module.prototype.init = function(handler, config) {};


Module.prototype.init = function (handler, config) {};

Module.prototype.setSerialPort = function (sp) {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.setSerialPort = function (sp) {
Module.prototype.setSerialPort = function(sp) {

Module.prototype.init = function (handler, config) {};

Module.prototype.setSerialPort = function (sp) {
var self = this;

Choose a reason for hiding this comment

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

⚠️ [eslint] <no-var> reported by reviewdog 🐶
Unexpected var, use let or const instead.

Suggested change
var self = this;
let self = this;

this.sp = sp;
};

Module.prototype.requestInitialData = function () {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.requestInitialData = function () {
Module.prototype.requestInitialData = function() {

self.digitalPortTimeList[dataObj.port] = dataObj.time;
}
} else if (Array.isArray(dataObj.port)) {
isSend = dataObj.port.every(function (port) {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
isSend = dataObj.port.every(function (port) {
isSend = dataObj.port.every(function(port) {

}
} else if (Array.isArray(dataObj.port)) {
isSend = dataObj.port.every(function (port) {
var time = self.digitalPortTimeList[port];

Choose a reason for hiding this comment

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

⚠️ [eslint] <no-var> reported by reviewdog 🐶
Unexpected var, use let or const instead.

Suggested change
var time = self.digitalPortTimeList[port];
let time = self.digitalPortTimeList[port];

});

if (isSend) {
dataObj.port.forEach(function (port) {

Choose a reason for hiding this comment

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

⚠️ [eslint] <prefer-arrow-callback> reported by reviewdog 🐶
Unexpected function expression.

Suggested change
dataObj.port.forEach(function (port) {
dataObj.port.forEach((port) => {

});

if (isSend) {
dataObj.port.forEach(function (port) {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
dataObj.port.forEach(function (port) {
dataObj.port.forEach(function(port) {

}

if (setDatas) {
var setKeys = Object.keys(setDatas);

Choose a reason for hiding this comment

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

⚠️ [eslint] <no-var> reported by reviewdog 🐶
Unexpected var, use let or const instead.

Suggested change
var setKeys = Object.keys(setDatas);
let setKeys = Object.keys(setDatas);

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.

8 participants