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

IStore.ListNamespaces() method #141

Merged
merged 1 commit into from Mar 21, 2019
Merged

Conversation

dahlia
Copy link
Contributor

@dahlia dahlia commented Mar 21, 2019

Although I frankly am not sure if it's really necessary, I added a new method ListNamespaces() to IStore and implemented FileSystem.ListNamespaces() too. I am going to use this to make an ad-hoc solution to #96.

@dahlia dahlia self-assigned this Mar 21, 2019
earlbread
earlbread previously approved these changes Mar 21, 2019
@codecov
Copy link

codecov bot commented Mar 21, 2019

Codecov Report

Merging #141 into master will decrease coverage by 2.73%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
- Coverage   87.68%   84.95%   -2.74%     
==========================================
  Files          69       69              
  Lines        3241     3250       +9     
==========================================
- Hits         2842     2761      -81     
- Misses        399      489      +90
Impacted Files Coverage Δ
Libplanet/Store/BaseStore.cs 100% <ø> (ø) ⬆️
Libplanet/Store/FileStore.cs 93.33% <100%> (+0.17%) ⬆️
Libplanet/Net/IceServer.cs 0% <0%> (-100%) ⬇️
Libplanet/Net/IceServerException.cs 0% <0%> (-100%) ⬇️
Libplanet/Net/NetworkStreamProxy.cs 0% <0%> (-80.77%) ⬇️
Libplanet/Net/Swarm.cs 85.07% <0%> (-4.51%) ⬇️

longfin
longfin previously approved these changes Mar 21, 2019
@dahlia
Copy link
Contributor Author

dahlia commented Mar 21, 2019

@longfin @earlbread I rebased this on the current master and added a changelog too. Could you review this again?

@dahlia dahlia merged commit 14a39fc into planetarium:master Mar 21, 2019
dahlia pushed a commit to dahlia/libplanet that referenced this pull request Mar 9, 2021
…Microsoft.NET.Test.Sdk-16.8.3

Bump Microsoft.NET.Test.Sdk from 15.9.0 to 16.8.3
OnedgeLee pushed a commit to OnedgeLee/libplanet that referenced this pull request Jan 31, 2023
…anifests-1669359664

Update internal manifests [update-internal-manifests-1669359664]
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.

None yet

3 participants