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

CI: package daily build kernel #285

Merged
merged 2 commits into from
Jun 17, 2024
Merged

CI: package daily build kernel #285

merged 2 commits into from
Jun 17, 2024

Conversation

opsiff
Copy link
Member

@opsiff opsiff commented Jun 17, 2024

No description provided.

name: package kernel amd64 daily
on:
schedule:
- cron: "0 2 * * *"

Choose a reason for hiding this comment

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

WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit description?)

- uses: actions/checkout@v3
- name: "Install Deps"
run: |
git config --global user.email $email

Choose a reason for hiding this comment

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

ERROR: trailing whitespace

- name: "Install Deps"
run: |
git config --global user.email $email
git config --global user.name $KBUILD_BUILD_USER

Choose a reason for hiding this comment

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

ERROR: trailing whitespace

name: package kernel amd64 daily
on:
schedule:
- cron: "0 2 * * *"

Choose a reason for hiding this comment

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

WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit description?)

- uses: actions/checkout@v3
- name: "Install Deps"
run: |
git config --global user.email $email

Choose a reason for hiding this comment

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

ERROR: trailing whitespace

- name: "Install Deps"
run: |
git config --global user.email $email
git config --global user.name $KBUILD_BUILD_USER

Choose a reason for hiding this comment

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

ERROR: trailing whitespace

@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 工作流文件命名应遵循项目规范,例如使用package-kernel-amd64-daily.yaml而不是package-kernel-amd64-daily.yml
  • 环境变量KBUILD_BUILD_USERKBUILD_BUILD_HOST的命名应遵循项目命名规范,避免与系统环境变量冲突
  • 缺少对actions/checkout@v3actions/upload-artifact@v3的版本检查或更新,以确保使用的是最新稳定版本
  • 缺少对nproc环境变量的定义或检查,可能会导致在某些系统上无法正确获取CPU核心数
  • 缺少对make命令的错误处理,例如在编译过程中出现错误时应该有相应的处理逻辑
  • 缺少对deepin_x86_desktop_defconfig配置的定义或说明,可能需要根据实际情况进行调整

是否建议立即修改:

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign xzl01 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

1 similar comment
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign xzl01 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@opsiff opsiff merged commit a1fa24b into linux-6.6.y Jun 17, 2024
7 of 9 checks passed
@opsiff opsiff deleted the package-kernel-daily branch June 17, 2024 01:49
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