Skip to content

[LAB1] 311555030 #34

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

Merged
merged 2 commits into from
Mar 7, 2024
Merged

[LAB1] 311555030 #34

merged 2 commits into from
Mar 7, 2024

Conversation

Jim890227
Copy link

Description

#2


  • A clear title (name your pr "[LAB{lab_number}] {your_student_id}")
  • A meaningful message for PR, as well as its commits
  • From your specific branch (not main or other's branch) merging to your branch
  • Excluding any irrelevant files, such as binaries, text files, or dot files
  • Passing tests/CI

@Jim890227 Jim890227 closed this Feb 29, 2024
@Jim890227 Jim890227 reopened this Feb 29, 2024
@TaiYou-TW TaiYou-TW closed this Feb 29, 2024
@TaiYou-TW TaiYou-TW reopened this Feb 29, 2024
@TaiYou-TW TaiYou-TW added the lab1 label Feb 29, 2024
@Jim890227
Copy link
Author

Jim890227 commented Feb 29, 2024 via email

@TaiYou-TW
Copy link
Contributor

助教您好,想請問一下那個test file每個test的最後一行是一定要在code中嗎?因為我發現上傳後好像會自動補上,(所以我先關閉確認上傳版本正確才又重新開啟),但是有那行會導致程式無法run,請問有需要做什麼更改嗎?

@Jim890227 同學你好:
不太確定你指的是哪個部分,是否是:

throw new Error("Test not implemented");

如果是指這個的話,我們本來就預期你應該將這個部分刪除,並補上你自己實作的 Unit Testing,所以沒有刪除會導致 CI 失敗是正常的。
另外提到上傳後會自動補上應該是不會發生的,我目前看你的 commit 也沒有發生,可能是你之前操作失誤或其他原因,如果想確定的話可以在 Lab 時間來了解詳細原因。

@Jim890227
Copy link
Author

Jim890227 commented Mar 7, 2024 via email

@TaiYou-TW TaiYou-TW merged commit 6b33cd4 into SQLab:311555030 Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants