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.

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

Development enable lint #140

Merged
merged 56 commits into from
Mar 15, 2021
Merged

Development enable lint #140

merged 56 commits into from
Mar 15, 2021

Conversation

y2468101216
Copy link
Collaborator

👏 解決掉的 issue / Resolved Issues

⛏ 變更內容 / Details of Changes

二合一

y2468101216 and others added 30 commits February 22, 2021 11:43
# Conflicts:
#	internal/delivery/http/route_users_test.go
# Conflicts:
#	internal/delivery/http/route_users_mock_test.go
merge development to development_enable_lint
修正 enable_lint branch 的 Google Action steps
…le_test

Test/#74add route boards article test
修正無用函數跟簡化宣告
修正了除了 interface 跟 mock 以外的 id
# Conflicts:
#	.github/workflows/go.yaml
#	internal/usecase/board.go
2021/02/22 ~2021/03/01 development to development_enable_lint
DeanXu2357 and others added 25 commits March 3, 2021 00:41
# Conflicts:
#	internal/delivery/http/route.go
#	internal/delivery/http/route_boards_mock_test.go
#	internal/delivery/http/route_classes_mock_test.go
同步 development 跟 development_enable_lint
…easures_test

實作  #route board treasures test
# Conflicts:
#	internal/delivery/http/route_users.go
#	internal/delivery/http/route_users_test.go
@y2468101216 y2468101216 self-assigned this Mar 15, 2021
@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (development@01436cf). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             development     #140   +/-   ##
==============================================
  Coverage               ?   36.06%           
==============================================
  Files                  ?       21           
  Lines                  ?      804           
  Branches               ?        0           
==============================================
  Hits                   ?      290           
  Misses                 ?      467           
  Partials               ?       47           

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 01436cf...a19272c. Read the comment docs.

Copy link
Member

@PichuChen PichuChen left a comment

Choose a reason for hiding this comment

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

LGTM

@PichuChen PichuChen merged commit 6239761 into development Mar 15, 2021
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.

[主線] [PTT] 將 developement 與 development_enable_lint 合而為一
6 participants