Skip to content

修改计算机专业导论和高级语言程序设计两门课程映射的错误#8

Open
WDGaster703 wants to merge 1 commit intomainfrom
WDGaster703-patch-1
Open

修改计算机专业导论和高级语言程序设计两门课程映射的错误#8
WDGaster703 wants to merge 1 commit intomainfrom
WDGaster703-patch-1

Conversation

@WDGaster703
Copy link
Contributor

📝 描述

@kowyo 群里提到的课程已修改,此PR可以等后续其他错误校对出来一并修改后再合并,如果要合并就直接合。

✅ 检查清单

  • 我已知晓并认同《参与指南》中的公约

  • 是否在合适的文件夹上传?

  • 是否获得了必要的许可?
  • 文件命名是否符合规范?

📢 通知成员

🤔 附加信息

@gemini-code-assist
Copy link

Summary of Changes

Hello @WDGaster703, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

此拉取请求旨在通过更新课程映射配置文件,解决特定课程代码与其默认关联课程之间的不准确问题。这些调整确保了系统能够正确识别和处理“计算机专业导论”和“高级语言程序设计”等课程的关联关系,从而提升了数据的一致性和准确性。

Highlights

  • 课程映射修正: 修正了 src/hoa_cli/data/lookup_table.toml 文件中计算机专业导论和高级语言程序设计相关课程的映射错误。
  • 映射关系更新: 调整了 COMP1014COMP2018COMP2021COMP1013 等课程的默认映射目标,以反映正确的课程关联。

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog
  • src/hoa_cli/data/lookup_table.toml
    • 新增了 COMP1014 的默认映射,指向 COMP2001
    • 新增了 COMP2018 的默认映射,指向 COMP1011
    • COMP2021 的默认映射从 COMP2001 更改为 COMP1011
    • COMP1013 的默认映射从 COMP2021 更改为 COMP1011
    • 移除了旧的 COMP1011COMP2021 的映射。
Activity
  • 拉取请求由 WDGaster703 创建,旨在修正课程映射错误。
  • 作者提及此修改是针对 @kowyo 在群里提到的问题。
  • 作者表示此PR可以等待后续其他错误校对后一并合并,或直接合并。
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

您好,感谢您的贡献!本次 PR 修正了 计算机专业导论高级语言程序设计 两门课程的错误映射,将相关的课程代码指向了正确的仓库 ID。我注意到配置文件中的条目可以按字母顺序排序以提高可维护性,并已在具体的评论中提出了建议。总体而言,这是一个很好的修复!

Comment on lines +85 to +95
[COMP1014]
DEFAULT = "COMP2001"

[COMP1011]
DEFAULT = "COMP2021"
[COMP2018]
DEFAULT = "COMP1011"

[COMP2021]
DEFAULT = "COMP1011"

[COMP1013]
DEFAULT = "COMP2021"
DEFAULT = "COMP1011"

Choose a reason for hiding this comment

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

medium

为了提高配置文件的可读性和可维护性,建议按课程代码对 COMP 相关条目进行字母排序。这样可以使文件结构更清晰,方便未来查找和修改。

Suggested change
[COMP1014]
DEFAULT = "COMP2001"
[COMP1011]
DEFAULT = "COMP2021"
[COMP2018]
DEFAULT = "COMP1011"
[COMP2021]
DEFAULT = "COMP1011"
[COMP1013]
DEFAULT = "COMP2021"
DEFAULT = "COMP1011"
[COMP1013]
DEFAULT = "COMP1011"
[COMP1014]
DEFAULT = "COMP2001"
[COMP2018]
DEFAULT = "COMP1011"
[COMP2021]
DEFAULT = "COMP1011"

@kowyo
Copy link
Member

kowyo commented Feb 17, 2026

@WDGaster703 现在改到 hoa-majors-data 那个仓库了

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