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.

Sign up for GitHub

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

Jump to bottom

fix unit tests #4

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

fix unit tests #4

wants to merge 7 commits into from

Conversation

WindzCUHK
Copy link
Contributor

@WindzCUHK WindzCUHK commented Dec 19, 2022 •
edited
Loading

Description

fix unit tests

Type of change

  • Bug fix
  • New feature
  • Refactoring (no functional changes, no api changes)
  • Non-code changes (update documentation, pipeline, etc.)

Flags

  • Breaks backward compatibility
  • Requires a documentation update
  • Has untestable code

Related issue/PR

Checklist

  • Followed the guidelines in the CONTRIBUTING document
  • Added prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip] in the PR title if necessary
  • Tested and linted the code
  • Commented the code
  • Made corresponding changes to the documentation
  • Passed all pipeline checking

Checklist for maintainer

  • Use Squash and merge
  • Double-confirm the merge message has prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip]
  • Delete the branch after merge

@WindzCUHK WindzCUHK force-pushed the fix_testcase branch 2 times, most recently from 27cabfb to ae99556 Compare December 19, 2022 03:27
kevindiu and others added 5 commits December 19, 2022 13:13
@kevindiu @WindzCUHK
fix
e3fab62 
Signed-off-by: wfan <[email protected]>
@kevindiu @WindzCUHK
fix
40d52b2 
Signed-off-by: wfan <[email protected]>
@kevindiu @WindzCUHK
fix
4be325f 
Signed-off-by: wfan <[email protected]>
@WindzCUHK
fix err msg, fix test case
7c8e5cf 
Signed-off-by: wfan <[email protected]>
@kevindiu @WindzCUHK
fix
05a08c8 
Signed-off-by: wfan <[email protected]>
@WindzCUHK WindzCUHK mentioned this pull request Dec 19, 2022
Closed
@mlajkim mlajkim added the refactor Changes without functional changes, or api changes) label Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
refactor Changes without functional changes, or api changes)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@WindzCUHK @mlajkim @kevindiu