Skip to content

Conversation

@nicewook
Copy link
Contributor

@nicewook nicewook commented Jun 6, 2021

  • client/main.go can test this GRPC function and save response as .pbtxt

- client/main.go can test this GRPC function and save response as .pbtxt
nicewook and others added 2 commits June 6, 2021 23:32
nicewook and others added 4 commits June 6, 2021 23:38
Co-authored-by: Mo Kweon <kkweon@gmail.com>
Co-authored-by: Mo Kweon <kkweon@gmail.com>
Co-authored-by: Mo Kweon <kkweon@gmail.com>
@nicewook
Copy link
Contributor Author

nicewook commented Jun 6, 2021

  1. 말씀하신 부분 참고하여 반영하였습니다.
  2. golangci-lint 에서 no such linter "godox" 라는 에러가 나면서 린트가 안되네요. 집 컴터에서만 안되는데 이거 이유를 모르겠네요
  3. evans 는 좀더 찾아보겠습니다. 리뷰 감사합니다.

Copy link
Member

@kkweon kkweon left a comment

Choose a reason for hiding this comment

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

아주 멋지네요. 👏 👏 👏

린트에러 및 비디오 객체 생성 관련 수정만 해주시면 머지하겠습니다.

nicewook and others added 2 commits June 7, 2021 10:24
Co-authored-by: Mo Kweon <kkweon@gmail.com>
@codecov
Copy link

codecov bot commented Jun 7, 2021

Codecov Report

Merging #55 (dd21eb8) into main (707eeec) will decrease coverage by 11.29%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##             main      #55       +/-   ##
===========================================
- Coverage   88.70%   77.41%   -11.30%     
===========================================
  Files           3        3               
  Lines          62       31       -31     
===========================================
- Hits           55       24       -31     
  Misses          6        6               
  Partials        1        1               
Impacted Files Coverage Δ
server/pkg/serv/serv.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f070abc...dd21eb8. Read the comment docs.

@kkweon kkweon merged commit 516d334 into main Jun 7, 2021
@kkweon kkweon deleted the feature/serve_getvideos branch June 7, 2021 03:51
@kkweon
Copy link
Member

kkweon commented Jun 7, 2021

수고하셨습니다

@nicewook
Copy link
Contributor Author

nicewook commented Jun 7, 2021

감사합니다. (__)

@github-actions
Copy link

github-actions bot commented Jun 8, 2021

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

3 participants