Skip to content

fix(course): make exam info field nullable - #526

Merged
ACaiCat merged 1 commit into
west2-online:mainfrom
CutebreadCat:fix/exam_field
Aug 8, 2026
Merged

fix(course): make exam info field nullable#526
ACaiCat merged 1 commit into
west2-online:mainfrom
CutebreadCat:fix/exam_field

Conversation

@CutebreadCat

Copy link
Copy Markdown
Contributor

自查 PR 结构

  • PR 标题符合这个格式: <type>(optional scope): <description>

  • 此 PR 标题的描述以用户为导向,足够清晰,其他人可以理解。

  • 我已经对所有 commit 提供了签名(GPG 密钥签名、SSH 密钥签名)

  • 这个 PR 属于强制变更/破坏性更改

如果是,请在 PR 标题中添加 BREAKING CHANGE 前缀,并在 PR 描述中详细说明。

这个 PR 的类型是什么?

fix

这个 PR 做了什么 / 我们为什么需要这个 PR?

修复了exam_info_sha256字段 null与json的问题

(可选)这个 PR 解决了哪个/些 issue?

对 Reviewer 预留的一些提醒

@codecov

codecov Bot commented Aug 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/course/service/get_course_list.go 60.00% 1 Missing and 1 partial ⚠️
@@            Coverage Diff             @@
##             main     #526      +/-   ##
==========================================
- Coverage   65.49%   64.10%   -1.39%     
==========================================
  Files         241      241              
  Lines        6529     6581      +52     
==========================================
- Hits         4276     4219      -57     
- Misses       2095     2204     +109     
  Partials      158      158              
Flag Coverage Δ
unittest 64.10% <60.00%> (-1.39%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
internal/course/service/get_course_list.go 75.75% <60.00%> (-0.54%) ⬇️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ACaiCat ACaiCat changed the title fix :fix exam_sha258 null field problem fix(course): exam_sha258 null field Aug 8, 2026
@ACaiCat ACaiCat changed the title fix(course): exam_sha258 null field fix(course): make exam info nullable Aug 8, 2026
@ACaiCat ACaiCat changed the title fix(course): make exam info nullable fix(course): make exam info field nullable Aug 8, 2026
@ACaiCat
ACaiCat merged commit 8a3cc7a into west2-online:main Aug 8, 2026
6 of 7 checks passed
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