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

删除第三方图片 #69

Closed
wants to merge 1 commit into from
Closed

删除第三方图片 #69

wants to merge 1 commit into from

Conversation

ChinaGodMan
Copy link
Owner

@ChinaGodMan ChinaGodMan commented Dec 23, 2024

变更内容

删除第三方检测的图片

变更类型

  • 修复 Bug
  • 新功能
  • 代码优化
  • 文档更新
  • 其他(请描述)
  • QA

Summary by Sourcery

移除第三方许可证徽章并更新文档中的语言名称。

文档:

  • 更新语言名称为“繁体中文”、“日本语”。
  • 移除FOSSA状态徽章和许可证部分。
Original summary in English

Summary by Sourcery

Remove the third-party license badge and update the language names in the documentation.

Documentation:

  • Update language names to "繁体中文", "日本语".
  • Remove the FOSSA status badge and license section.

Copy link

changeset-bot bot commented Dec 23, 2024

⚠️ No Changeset found

Latest commit: f8c0128

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

pr-code-reviewer bot commented Dec 23, 2024

👋 Hi there!

The title isn't very descriptive.


Automatically generated with the help of gpt-3.5-turbo.
Feedback? Please don't hesitate to drop me an email at webber@takken.io.

Copy link

instapr bot commented Dec 23, 2024

Comment

The image removal looks good. No issues found with the changes made in the PR. Feel free to merge. 👍

- [x] QA

@labels-and-badges labels-and-badges bot added NO JIRA This PR does not have a Jira Ticket PR:size/XS Denotes a Pull Request that changes 0-9 lines. release This PR is a release labels Dec 23, 2024
Copy link
Contributor

@ChinaGodMan 你好,人民的勤务员将尽快审查合并此次请求!🚀 [自动回复,请勿跟帖]

Copy link

sourcery-ai bot commented Dec 23, 2024

审核指南 by Sourcery

此拉取请求移除了第三方图像验证徽章,并将语言名称更新为其本地拼写。

未生成图表,因为更改看起来很简单,不需要视觉表示。

文件级更改

更改 详情 文件
移除了 FOSSA 许可证徽章及其相关链接。
  • 从 README 文件底部删除了 FOSSA 状态图像和链接。
docs/README.md
更新了标题中的语言名称以使用本地拼写。
  • 将 "繁體中文" 更改为 "繁体中文"
  • 将 "日本語" 更改为 "日本语"
docs/README.md

提示和命令

与 Sourcery 互动

  • 触发新审核: 在拉取请求中评论 @sourcery-ai review
  • 继续讨论: 直接回复 Sourcery 的审核评论。
  • 从审核评论生成 GitHub 问题: 通过回复审核评论请求 Sourcery 创建一个问题。
  • 生成拉取请求标题: 在拉取请求标题的任何地方写 @sourcery-ai 以随时生成标题。
  • 生成拉取请求摘要: 在拉取请求正文的任何地方写 @sourcery-ai summary 以随时生成 PR 摘要。您也可以使用此命令指定摘要应插入的位置。

自定义您的体验

访问您的仪表板以:

  • 启用或禁用审核功能,例如 Sourcery 生成的拉取请求摘要、审核指南等。
  • 更改审核语言。
  • 添加、删除或编辑自定义审核说明。
  • 调整其他审核设置。

获取帮助

Original review guide in English

Reviewer's Guide by Sourcery

This pull request removes the third-party image verification badge and updates the language names to their native spellings.

No diagrams generated as the changes look simple and do not need a visual representation.

File-Level Changes

Change Details Files
Removed the FOSSA license badge and its related link.
  • Deleted the FOSSA Status image and link from the bottom of the README file.
docs/README.md
Updated language names in the header to use native spellings.
  • Changed "繁體中文" to "繁体中文"
  • Changed "日本語" to "日本语"
docs/README.md

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time. You can also use
    this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link

@llamapreview llamapreview bot left a comment

Choose a reason for hiding this comment

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

Auto Pull Request Review from LlamaPReview

Review Status: Automated Review Skipped

Dear contributor,

Thank you for your Pull Request. LlamaPReview has analyzed your changes and determined that this PR does not require an automated code review.

Analysis Result:

PR only contains documentation changes (1 files)

Technical Context:

Documentation changes typically include:

  • Markdown/RST file updates
  • API documentation
  • Code comments
  • README updates
  • Documentation in /docs directory
  • License and contribution files

We're continuously improving our PR analysis capabilities. Have thoughts on when and how LlamaPReview should perform automated reviews? Share your insights in our GitHub Discussions.

Best regards,
LlamaPReview Team

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

@ChinaGodMan - 我已经审查了你的更改 - 这里有一些反馈:

总体评论

  • 一些语言名称无意中从繁体中文更改为简体中文(例如,“繁體中文”到“繁体中文”)。请确认此更改是否是有意的。
这是我在审查期间查看的内容
  • 🟡 一般问题:发现1个问题
  • 🟢 安全性:一切看起来都很好
  • 🟢 测试:一切看起来都很好
  • 🟢 复杂性:一切看起来都很好
  • 🟢 文档:一切看起来都很好

Sourcery对开源项目免费 - 如果你喜欢我们的评论,请考虑分享它们 ✨
帮助我变得更有用!请点击每条评论上的👍或👎,我将使用反馈来改进你的评论。
Original comment in English

Hey @ChinaGodMan - I've reviewed your changes - here's some feedback:

Overall Comments:

  • Some language names have been unintentionally changed from traditional to simplified Chinese characters (e.g., '繁體中文' to '繁体中文'). Please verify if this change was intended.
Here's what I looked at during the review
  • 🟡 General issues: 1 issue found
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

<a href="https://github.com/ChinaGodMan/UserScripts/blob/main/docs/en/README.md">English</a> |
<a href="https://github.com/ChinaGodMan/UserScripts/blob/main/docs/ja/README.md">日本語</a>|
<a href="https://github.com/ChinaGodMan/UserScripts/blob/main/docs/ja/README.md">日本语</a>|
Copy link

Choose a reason for hiding this comment

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

issue (typo): 日语翻译错误。

日语翻译应为“日本語”而不是“日本语”。

Suggested change
<a href="https://github.com/ChinaGodMan/UserScripts/blob/main/docs/ja/README.md">日本语</a>|
<a href="https://github.com/ChinaGodMan/UserScripts/blob/main/docs/ja/README.md">日本語</a>|
Original comment in English

issue (typo): Incorrect Japanese translation.

The Japanese translation should be "日本語" instead of "日本语".

Suggested change
<a href="https://github.com/ChinaGodMan/UserScripts/blob/main/docs/ja/README.md">日本语</a>|
<a href="https://github.com/ChinaGodMan/UserScripts/blob/main/docs/ja/README.md">日本語</a>|

@ChinaGodMan ChinaGodMan deleted the del branch December 23, 2024 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NO JIRA This PR does not have a Jira Ticket PR:size/XS Denotes a Pull Request that changes 0-9 lines. release This PR is a release size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant