Skip to content

Implement default limit#174

Merged
groenroos merged 1 commit intomasterfrom
feature/default-limit
Apr 4, 2021
Merged

Implement default limit#174
groenroos merged 1 commit intomasterfrom
feature/default-limit

Conversation

@groenroos
Copy link
Member

Closes #164.

@groenroos groenroos added the feature Brand new features to Sapling label Apr 4, 2021
@groenroos groenroos added this to the M2 - Release 2.0 milestone Apr 4, 2021
@groenroos groenroos self-assigned this Apr 4, 2021
@codecov
Copy link

codecov bot commented Apr 4, 2021

Codecov Report

Merging #174 (f0369d8) into master (e8b0a1c) will decrease coverage by 0.20%.
The diff coverage is 40.00%.

❗ Current head f0369d8 differs from pull request most recent head 832d525. Consider uploading reports for the commit 832d525 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #174      +/-   ##
==========================================
- Coverage   91.59%   91.39%   -0.21%     
==========================================
  Files          35       35              
  Lines        1238     1243       +5     
==========================================
+ Hits         1134     1136       +2     
- Misses        104      107       +3     
Impacted Files Coverage Δ
core/loadConfig.js 100.00% <ø> (ø)
lib/Storage.js 82.59% <40.00%> (-0.88%) ⬇️

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 e8b0a1c...832d525. Read the comment docs.

@groenroos groenroos merged commit f48a949 into master Apr 4, 2021
@groenroos groenroos deleted the feature/default-limit branch April 4, 2021 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Brand new features to Sapling

Development

Successfully merging this pull request may close these issues.

Add default limit for db reads

1 participant