Skip to content
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

[1.9.49] 버전업 PR #698

Merged
merged 15 commits into from
Dec 23, 2023
Merged

[1.9.49] 버전업 PR #698

merged 15 commits into from
Dec 23, 2023

Conversation

Tnks2U
Copy link
Contributor

@Tnks2U Tnks2U commented Dec 23, 2023

@Tnks2U Tnks2U self-assigned this Dec 23, 2023
@Tnks2U Tnks2U merged commit 1125816 into master Dec 23, 2023
1 check passed
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/c3coding_arduino.js|175 col 29| Unexpected function expression.
app/modules/c3coding_arduino.js|175 col 37| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|176 col 17| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|210 col 28| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|211 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|212 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|215 col 13| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|216 col 13| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|217 col 55| Unexpected function expression.
app/modules/c3coding_arduino.js|217 col 63| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|218 col 17| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|234 col 9| Block must not be padded by blank lines.
app/modules/c3coding_arduino.js|250 col 32| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|251 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|255 col 53| Unexpected function expression.
app/modules/c3coding_arduino.js|255 col 61| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|257 col 35| Unexpected function expression.
app/modules/c3coding_arduino.js|257 col 43| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|269 col 31| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|270 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|271 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|273 col 23| Unexpected function expression.
app/modules/c3coding_arduino.js|273 col 31| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|277 col 13| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|278 col 13| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|295 col 13| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|296 col 13| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|325 col 36| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|326 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|327 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|352 col 13| 'value' is not defined.
app/modules/c3coding_arduino.js|353 col 13| 'value' is not defined.
app/modules/c3coding_arduino.js|364 col 45| 'value' is not defined.
app/modules/c3coding_arduino.js|375 col 32| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|376 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|377 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|378 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|397 col 17| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|423 col 31| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|424 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|425 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|426 col 24| Unexpected function expression.
app/modules/c3coding_arduino.js|426 col 32| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|436 col 26| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|437 col 9| Unexpected var, use let or const instead.
app/modules/c3coding_arduino.js|444 col 21| Unexpected space before function parentheses.
app/modules/c3coding_arduino.js|451 col 22| A constructor name should not start with a lowercase letter.
app/modules/c3coding_arduino.js|451 col 41| Newline required at end of file but not found.
app/modules/hexaboard.js|308 col 1| This line has a length of 137. Maximum allowed is 120.
app/modules/hexaboard.js|729 col 22| A constructor name should not start with a lowercase letter.

@@ -0,0 +1,451 @@
const BaseModule = require('./baseModule');

class c3coding_arduino extends BaseModule {

Choose a reason for hiding this comment

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

⚠️ [eslint] <camelcase> reported by reviewdog 🐶
Identifier 'c3coding_arduino' is not in camel case.

this.handler = handler;
this.config = config;
}
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
setSerialPort = function (sp) {
setSerialPort = function(sp) {

this.config = config;
}
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;
};
// 연결 후 초기에 송신할 데이터가 필요한 경우 사용합니다.(필수)
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
requestInitialData = function () {
requestInitialData = function() {

return this.makeSensorReadBuffer(this.sensorTypes.ANALOG, 0);
};
// 연결 후 초기에 수신받아서 정상연결인지를 확인해야하는 경우 사용합니다.(필수)
checkInitialData = function (data, 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
checkInitialData = function (data, config) {
checkInitialData = function(data, config) {

}
} 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) {

self.makeSensorReadBuffer(
key,
dataObj.port,
dataObj.data

Choose a reason for hiding this comment

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

⚠️ [eslint] <comma-dangle> reported by reviewdog 🐶
Missing trailing comma.

Suggested change
dataObj.data
dataObj.data,

}

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.

5 participants