Skip to content

Conversation

@jxs1211
Copy link

@jxs1211 jxs1211 commented Jul 26, 2022

Pre-Checklist

Note: Please complete ALL items in the following checklist.

  • I have read through the CONTRIBUTING.md documentation.
  • My code has the necessary comments and documentation (if needed).
  • I have added relevant tests

Description

add ut for download

Related Issues

see #833 for the details

New Behavior (screenshots if needed)

github.com/devstream-io/devstream/pkg/util/github/download.go:14: DownloadAsset 100.0%
github.com/devstream-io/devstream/pkg/util/github/download.go:83: DownloadLatestCodeAsZipFile 100.0%

@jxs1211 jxs1211 requested a review from a team as a code owner July 26, 2022 07:10
@jxs1211 jxs1211 force-pushed the feat-pkg-util-github-download branch from 4e0c37f to f35a793 Compare July 26, 2022 09:51
@xavier-hou
Copy link
Member

@jxs1211 Many thanks for your test! Please fix the redundant commits~

@jxs1211 jxs1211 force-pushed the feat-pkg-util-github-download branch from f35a793 to a9cb764 Compare July 27, 2022 10:51
@jxs1211
Copy link
Author

jxs1211 commented Jul 27, 2022

@jxs1211 Many thanks for your test! Please fix the redundant commits~

done

@aFlyBird0 aFlyBird0 removed their assignment Jul 28, 2022
@aFlyBird0
Copy link
Member

aFlyBird0 commented Jul 28, 2022

I have checked the format of the test, could you please check the content of the test? Thanks! @HXCGIT

@aFlyBird0
Copy link
Member

@jxs1211 Please remember to resolve the conflicts. 🙌

Signed-off-by: xian-jie.shen <327411586@qq.com>
@jxs1211 jxs1211 force-pushed the feat-pkg-util-github-download branch from a9cb764 to d3149cf Compare August 1, 2022 12:09
@jxs1211
Copy link
Author

jxs1211 commented Aug 1, 2022

@jxs1211 Please remember to resolve the conflicts. 🙌
resolved

@xavier-hou
Copy link
Member

@jxs1211 Nice test!

@xavier-hou xavier-hou assigned aFlyBird0 and unassigned xavier-hou Aug 1, 2022
Copy link
Member

@aFlyBird0 aFlyBird0 left a comment

Choose a reason for hiding this comment

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

Excellent job! 👏

@aFlyBird0 aFlyBird0 merged commit 9f2dd41 into devstream-io:main Aug 1, 2022
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.

3 participants