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

feat:Post action 中支持获取父任务ID #10968 #11001

Closed
wants to merge 3 commits into from

Conversation

Kzhou991
Copy link
Collaborator

No description provided.

@tencentblueking-adm
Copy link

tencentblueking-adm commented Sep 23, 2024

CLA assistant check
All committers have signed the CLA.

@@ -58,6 +58,8 @@ const val CLOSE = "close" // 开关关闭
const val ATOM_POST = "post" // 插件post
const val ATOM_POST_FLAG = "postFlag" // 插件post标识
const val ATOM_POST_ENTRY_PARAM = "postEntryParam" // 插件post操作入口参数
const val ATOM_POST_PARENT_TASK_ID = "postActionParentTaskId" // 插件后置动作父任务id
Copy link
Collaborator

Choose a reason for hiding this comment

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

postActionParentTaskId换成parentTaskId

@@ -256,7 +259,7 @@ open class MarketAtomTask : ITask() {
// 开关关闭则不再写入插件私有配置到input.json中
inputVariables.putAll(getAtomSensitiveConfMap(atomCode))
}
writeInputFile(atomTmpSpace, inputVariables)
// writeInputFile(atomTmpSpace, inputVariables)
Copy link
Collaborator

Choose a reason for hiding this comment

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

没用的注释去掉

@@ -151,6 +153,7 @@ open class MarketAtomTask : ITask() {
val map = JsonUtil.toMutableMap(data)
// 该插件执行的工作空间绝对路径
val workspacePath = workspace.absolutePath
val additionalOptions = taskParams["additionalOptions"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个变量定义这么靠前干什么,另外"additionalOptions"换成Element::additionalOptions.name更加优雅

@Kzhou991 Kzhou991 changed the title Issue 10968 feat:Post action 中支持获取父任务ID #10968 Sep 23, 2024
@Kzhou991 Kzhou991 closed this Sep 23, 2024
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.

3 participants