Skip to content

1:移除工程使用lodash包时生成的.LICENSE.txt文件。2:上报开发者使用编辑器 - #172

Open
oneyouzi wants to merge 11 commits into
hapjs-platform:mainfrom
oneyouzi:main
Open

oneyouzi wants to merge 11 commits into
hapjs-platform:mainfrom
oneyouzi:main

Conversation

@oneyouzi

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread packages/hap-debugger/src/utils.js Outdated
}
// 使用环境变量判断是在cursor里面打开的
if (
(env.VSCODE_GIT_ASKPASS_NODE && env.VSCODE_GIT_ASKPASS_NODE.includes('cursor')) ||

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

这里 ai 审查了下,includes('cursor')) 这里不区分大小写有些问题。

VSCODE_GIT_ASKPASS_NODE 变量是 编辑器 注入的软件安装路径(https://github.com/microsoft/vscode/blob/94a39f4cb65288f75dfeeb9a4aa0e7146244e33c/extensions/git/src/askpass.ts#L40),但同时 cursor 在 macos 上的安装路径用的是 “Cursor”。

所以对于 macos cursor,落盘直接进到 other

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已修复

Comment thread packages/hap-debugger/src/utils.js Outdated
}

/**
* 判断当前工程是在哪一个ide里面打开的

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

不是那么严谨,看起来不是 “判断当前工程是在哪一个ide里面打开的”,而是 “判断当前 hap-toolkit 在哪里编译工程”

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已更正

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