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

feat(server): notify plugin donwload to marketplace #341

Merged
merged 1 commit into from Sep 20, 2022

Conversation

rot1024
Copy link
Member

@rot1024 rot1024 commented Sep 16, 2022

Overview

What I've done

What I haven't done

How I tested

Which point I want you to review particularly

Memo

@codecov
Copy link

codecov bot commented Sep 16, 2022

Codecov Report

Merging #341 (0645b16) into main (b521326) will increase coverage by 0.00%.
The diff coverage is 52.08%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #341   +/-   ##
=======================================
  Coverage   35.72%   35.73%           
=======================================
  Files         339      339           
  Lines       30202    30239   +37     
=======================================
+ Hits        10789    10805   +16     
- Misses      18369    18384   +15     
- Partials     1044     1050    +6     
Impacted Files Coverage Δ
server/internal/app/config.go 48.29% <ø> (ø)
server/internal/app/repo.go 0.00% <0.00%> (ø)
...erver/internal/usecase/interactor/plugin_common.go 16.66% <0.00%> (-1.29%) ⬇️
...internal/infrastructure/marketplace/marketplace.go 57.74% <58.13%> (-9.83%) ⬇️

@rot1024 rot1024 merged commit 59f8a85 into main Sep 20, 2022
@rot1024 rot1024 deleted the notify-plugin-download branch September 20, 2022 01:10
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.

None yet

1 participant