-
Notifications
You must be signed in to change notification settings - Fork 73
chore: rename ide in readme #1126
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
Conversation
change deepin-unioncode to deepin-ide Log: Change-Id: Iedf7266fdbacf152eb9d5cb50085a365d2101862
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR renames the project from "deepin-unioncode" to "deepin-ide" by updating all relevant references, build and usage instructions, and cleaning up outdated links in both the Chinese and English README files. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey there - I've reviewed your changes and they look great!
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location> `README.en.md:78` </location>
<code_context>
-Execute `deepin-unioncode`
+Execute `deepin-ide`
*Attention: After the installation of deepin-unicode, some features require the installation of dependency packages before they can be used normally.*
</code_context>
<issue_to_address>
**issue (typo):** Typo: 'deepin-unicode' should be 'deepin-ide'.
Update the reference to 'deepin-ide' for consistency with the documentation.
```suggestion
*Attention: After the installation of deepin-ide, some features require the installation of dependency packages before they can be used normally.*
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| Execute `deepin-unioncode` | ||
| Execute `deepin-ide` | ||
|
|
||
| *Attention: After the installation of deepin-unicode, some features require the installation of dependency packages before they can be used normally.* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue (typo): Typo: 'deepin-unicode' should be 'deepin-ide'.
Update the reference to 'deepin-ide' for consistency with the documentation.
| *Attention: After the installation of deepin-unicode, some features require the installation of dependency packages before they can be used normally.* | |
| *Attention: After the installation of deepin-ide, some features require the installation of dependency packages before they can be used normally.* |
|
Note
详情{
"README.md": [
{
"line": "$ git clone https://github.com/linuxdeepin/deepin-ide.git ",
"line_number": 54,
"rule": "S35",
"reason": "Url link | 0ba5d61089"
},
{
"line": "(gitee: $ git clone https://gitee.com/deepin-community/deepin-ide.git)",
"line_number": 55,
"rule": "S35",
"reason": "Url link | 9facf66e85"
},
{
"line": "| deepin-draw | 深度画板应用 | https://github.com/linuxdeepin/deepin-draw<br />(https://gitee.com/deepin-community/deepin-draw.git) |",
"line_number": 94,
"rule": "S35",
"reason": "Url link | e4cd920ca9"
},
{
"line": "| deepin-image-viewer | 深度看图应用 | https://github.com/linuxdeepin/deepin-image-viewer<br />(https://gitee.com/deepin-community/deepin-image-viewer.git) |",
"line_number": 95,
"rule": "S35",
"reason": "Url link | 7b68c4b580"
},
{
"line": "| deepin-album | 深度相册应用 | https://github.com/linuxdeepin/deepin-album<br />(https://gitee.com/deepin-community/deepin-album.git) |",
"line_number": 96,
"rule": "S35",
"reason": "Url link | c05ac56e05"
}
],
"README.en.md": [
{
"line": "$ git clone https://github.com/linuxdeepin/deepin-ide.git",
"line_number": 53,
"rule": "S35",
"reason": "Url link | 0ba5d61089"
},
{
"line": "(gitee: $ git clone https://gitee.com/deepin-community/deepin-ide.git)",
"line_number": 54,
"rule": "S35",
"reason": "Url link | 9facf66e85"
},
{
"line": "| deepin-draw | A lightweight drawing tool for users to freely draw | https://github.com/linuxdeepin/deepin-draw<br />(https://gitee.com/deepin-community/deepin-draw.git) |",
"line_number": 93,
"rule": "S35",
"reason": "Url link | e4cd920ca9"
},
{
"line": "| deepin-image-viewer | Image viewing tool | https://github.com/linuxdeepin/deepin-image-viewer<br />(https://gitee.com/deepin-community/deepin-image-viewer.git) |",
"line_number": 94,
"rule": "S35",
"reason": "Url link | 7b68c4b580"
},
{
"line": "| deepin-album | A photo management tool | https://github.com/linuxdeepin/deepin-album<br />(https://gitee.com/deepin-community/deepin-album.git) |",
"line_number": 95,
"rule": "S35",
"reason": "Url link | c05ac56e05"
}
]
} |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deepin-mozart The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
deepin pr auto review根据提供的git diff,我看到这是一个将项目名称从"deepin-unioncode"改为"deepin-ide"的修改。以下是我的审查意见:
建议改进:
总体来说,这次修改是合理的,主要是项目名称的统一更新。但需要注意确保所有相关文档和配置都已同步更新。 |
|
/forcemerge |
|
This pr force merged! (status: blocked) |
change deepin-unioncode to deepin-ide
Log:
Change-Id: Iedf7266fdbacf152eb9d5cb50085a365d2101862
Summary by Sourcery
Rename project from deepin-unioncode to deepin-ide and update documentation accordingly
Documentation: