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: add --alias option for repo add #986

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

ice909
Copy link
Collaborator

@ice909 ice909 commented Feb 13, 2025

ll-cli repo addll-builder repo add 添加--alias 选项以设置别名
同时update 、remove 和 set-default 也需要传入alias而不是name

@dengbo11
Copy link
Collaborator

dengbo11 commented Feb 13, 2025

下面两个小问题改下:

  1. yaml文件目前是采用的单引的方式,详见.editorconfig;
  2. 修一下如下类似地方的github actions警告;
    image

@dengbo11
Copy link
Collaborator

ll-cli repo addll-builder repo add 添加--alias 选项以设置别名 同时update 、remove 和 set-default 也需要传入alias而不是name

commit msg最好写英文,首字母大写,结尾加英文句号"."

@ice909 ice909 force-pushed the master branch 2 times, most recently from 1ab0d23 to aa5f604 Compare February 13, 2025 08:16
dengbo11
dengbo11 previously approved these changes Feb 13, 2025
apps/ll-builder/src/main.cpp Outdated Show resolved Hide resolved
apps/ll-builder/src/main.cpp Outdated Show resolved Hide resolved
libs/linglong/src/linglong/repo/config.cpp Outdated Show resolved Hide resolved
libs/linglong/src/linglong/repo/config.cpp Outdated Show resolved Hide resolved
misc/share/linglong/config.yaml Outdated Show resolved Hide resolved
@ice909 ice909 force-pushed the master branch 3 times, most recently from 0c3805f to 47ba8bc Compare February 13, 2025 12:17
Add the --alias option to set aliases for the 'll-cli repo add' and 'll-builder repo add' commands.

So update, remove, and set-default also need to pass alias instead of name.
@ComixHe ComixHe merged commit 4b3fed1 into OpenAtom-Linyaps:master Feb 13, 2025
5 of 13 checks passed
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