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

chore: Remove ActivityPub actor store #1357

Merged
merged 1 commit into from
Jun 16, 2022
Merged

Conversation

bstasyszyn
Copy link
Contributor

The actor store is not used.

closes #1356

Signed-off-by: Bob Stasyszyn Bob.Stasyszyn@securekey.com

The actor store is not used.

closes trustbloc#1356

Signed-off-by: Bob Stasyszyn <Bob.Stasyszyn@securekey.com>
@cla-bot cla-bot bot added the cla-signed label Jun 16, 2022
@codecov
Copy link

codecov bot commented Jun 16, 2022

Codecov Report

Merging #1357 (19b4cac) into main (062a592) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1357      +/-   ##
==========================================
- Coverage   87.81%   87.76%   -0.06%     
==========================================
  Files         199      199              
  Lines       18582    18546      -36     
==========================================
- Hits        16318    16277      -41     
  Misses       1457     1457              
- Partials      807      812       +5     
Impacted Files Coverage Δ
pkg/activitypub/store/memstore/memstore.go 94.28% <ø> (-0.46%) ⬇️
pkg/activitypub/store/spi/spi.go 25.00% <ø> (ø)
pkg/activitypub/store/ariesstore/ariesstore.go 88.23% <100.00%> (+0.41%) ⬆️
pkg/resolver/resource/resolver.go 90.80% <0.00%> (-9.20%) ⬇️
pkg/linkset/linkset.go 86.82% <0.00%> (-3.14%) ⬇️
pkg/vct/proofmonitoring/monitoring.go 84.07% <0.00%> (-2.66%) ⬇️
internal/pkg/ldcontext/ldcontext.go 61.53% <0.00%> (-1.62%) ⬇️
cmd/orb-server/startcmd/start.go 68.25% <0.00%> (+0.33%) ⬆️

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 062a592...19b4cac. Read the comment docs.

@fqutishat fqutishat merged commit e8822ee into trustbloc:main Jun 16, 2022
@bstasyszyn bstasyszyn deleted the 1356 branch June 30, 2022 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove ActivityPub Actor store
2 participants