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

fix(@feflow/cli): 修复开发套件命令注册错误 #420

Merged
merged 1 commit into from
Oct 22, 2022

Conversation

xqxian
Copy link
Collaborator

@xqxian xqxian commented Oct 22, 2022

用户使用 .feflowrc 注册项目命令时,例如:

{
  "projectCmd": {
    "builder" : "feflow-devkit-demo:devkitCmd"
  }
}

应读取 feflow-devkit-demo
devkit.jsondevkitCmd 对应的配置,而不是 projectCmd

@xqxian xqxian added the bug label Oct 22, 2022
@xqxian xqxian self-assigned this Oct 22, 2022
@tencent-adm
Copy link
Member

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


alexqxxu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

用户使用 `.feflowrc` 注册项目命令时,例如:
```json
{
  "projectCmd": {
    "builder" : "feflow-devkit-demo:devkitCmd"
  }
}
```
应读取 `feflow-devkit-demo` 下
`devkit.json` 中 `devkitCmd` 对应的配置,而不是 `projectCmd` 的
@xqxian xqxian force-pushed the bugfix/devkit_cmd_registration branch from 7b8ab98 to 5ce6f49 Compare October 22, 2022 09:19
@xqxian xqxian merged commit 41a747a into master Oct 22, 2022
@xqxian xqxian deleted the bugfix/devkit_cmd_registration branch October 22, 2022 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants