Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026

Plan to Remove .obs Directory

  • Remove the .obs directory and its contents (workflows.yml)
  • Update REUSE.toml to remove reference to .obs/**.yml
  • Update commit message format per review feedback

Summary

Successfully removed the .obs directory and its internal configuration. The OBS packaging is now triggered by the https://github.com/deepin-community/infra-settings project.

Changes:

  1. Deleted .obs/workflows.yml file
  2. Updated REUSE.toml to remove .obs/**.yml from annotations
  3. Updated commit message to follow conventional format with proper prefix, English and Chinese descriptions, and Influence section
Original prompt

删除 .obs 目录和内部的配置,现在触发obs打包是靠https://github.com/deepin-community/infra-settings项目,不再需要此配置文件


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@deepin-ci-robot
Copy link

Hi @Copilot. Thanks for your PR.

I'm waiting for a linuxdeepin member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Co-authored-by: zccrs <13449038+zccrs@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove .obs directory and its internal configuration Remove .obs directory - OBS packaging migrated to infra-settings Jan 15, 2026
Copilot AI requested a review from zccrs January 15, 2026 07:04
@zccrs zccrs marked this pull request as ready for review January 15, 2026 07:04
@zccrs
Copy link
Member

zccrs commented Jan 15, 2026

@copilot 我看commit信息的格式检查出错了,下面给你一个参考格式:

fix: Correct typo in variable name from wpModle to wpModel

Fixed variable name typo throughout the codebase where "wpModle"
was incorrectly spelled instead of "wpModel". This change ensures
consistency in variable naming and improves code readability.
Additionally, temporarily disabled the smart cascaded placement
functionality by adding an early return statement for further
development.

The typo correction affects multiple workspace-related functions
including surface placement, preview management, and active surface
tracking. While the variable name change is purely cosmetic, the early
return in placeSmartCascaded indicates this feature is being reworked.

Influence:
1. Verify workspace surface management functions work correctly after
variable name changes
2. Test surface placement and preview functionality
3. Check that active surface tracking operates as expected
4. Confirm no regression in workspace model operations

fix: 修正变量名拼写错误,将wpModle改为wpModel

修复了整个代码库中变量名拼写错误,"wpModle"被错误地拼写为"wpModel"。此更
改确保了变量命名的一致性并提高了代码可读性。此外,通过添加早期返回语句暂
时禁用了智能级联放置功能以便进一步开发。

拼写更正影响了多个工作区相关功能,包括表面放置、预览管理和活动表面跟踪。
虽然变量名称更改是纯外观上的,但placeSmartCascaded中的早期返回表明此功能
正在重新开发。

Influence:
1. 验证变量名称更改后工作区表面管理功能是否正常工作
2. 测试表面放置和预览功能
3. 检查活动表面跟踪是否按预期运行
4. 确认工作区模型操作没有回归

Copilot AI and others added 2 commits January 15, 2026 07:24
Removed the .obs directory containing legacy OBS CI/CD workflow
configuration as OBS packaging is now triggered by the
deepin-community/infra-settings project. The local .obs/workflows.yml
file is no longer needed for the build and packaging process.

Updated REUSE.toml to remove the .obs/**.yml pattern from license
annotations since the directory no longer exists in the repository.

This change eliminates obsolete configuration files and keeps the
repository clean by removing infrastructure configuration that has
been migrated to a centralized location.

Influence:
1. Verify that OBS packaging still works via infra-settings project
2. Confirm REUSE compliance checks pass without .obs references
3. Ensure no build or CI/CD processes depend on local .obs config

chore: 删除过时的 .obs 目录并更新 REUSE.toml

删除了包含旧版 OBS CI/CD 工作流配置的 .obs 目录,因为 OBS 打包现在由
deepin-community/infra-settings 项目触发。本地 .obs/workflows.yml 文件
不再需要用于构建和打包过程。

更新了 REUSE.toml 以从许可证注释中删除 .obs/**.yml 模式,因为该目录不
再存在于仓库中。

此更改消除了过时的配置文件,并通过删除已迁移到集中位置的基础设施配置来
保持仓库整洁。

Influence:
1. 验证 OBS 打包仍然通过 infra-settings 项目正常工作
2. 确认 REUSE 合规性检查在没有 .obs 引用的情况下通过
3. 确保没有构建或 CI/CD 流程依赖本地 .obs 配置

Co-authored-by: zccrs <13449038+zccrs@users.noreply.github.com>

Co-authored-by: zccrs <13449038+zccrs@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 15, 2026

@copilot 我看commit信息的格式检查出错了,下面给你一个参考格式:

fix: Correct typo in variable name from wpModle to wpModel...

Updated commit message format in commit 2b1b490. The message now follows the conventional format with chore: prefix, detailed English and Chinese descriptions, and Influence section listing verification points.

Copilot AI requested a review from zccrs January 15, 2026 07:26
@zccrs zccrs merged commit d51fe74 into master Jan 15, 2026
10 of 13 checks passed
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Copilot, zccrs

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

Details 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

@zccrs zccrs deleted the copilot/remove-obs-directory-and-config branch January 15, 2026 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants