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

fix: allow to use only prefix or namespace with hashmap #9

Merged
merged 1 commit into from Feb 5, 2020

Conversation

@vmarchaud
Copy link
Member

vmarchaud commented Feb 5, 2020

No description provided.

@vmarchaud vmarchaud requested a review from Eywek Feb 5, 2020
@vmarchaud vmarchaud self-assigned this Feb 5, 2020
@codecov

This comment has been minimized.

Copy link

codecov bot commented Feb 5, 2020

Codecov Report

Merging #9 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master      #9   +/-   ##
======================================
  Coverage    99.6%   99.6%           
======================================
  Files           9       9           
  Lines         507     507           
  Branches      118     119    +1     
======================================
  Hits          505     505           
  Misses          2       2
Impacted Files Coverage Δ
src/layers/redis.ts 98.69% <100%> (ø) ⬆️

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 a38de3d...b5f4d3c. Read the comment docs.

@Eywek
Eywek approved these changes Feb 5, 2020
@Eywek Eywek merged commit 6529f6c into master Feb 5, 2020
4 checks passed
4 checks passed
ci/circleci: lint Your tests passed on CircleCI!
Details
ci/circleci: test Your tests passed on CircleCI!
Details
codecov/patch 100% of diff hit (target 99.6%)
Details
codecov/project 99.6% (+0%) compared to a38de3d
Details
@Eywek Eywek deleted the fix/allow-no-prefix-hash branch Feb 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.