search

digitalbazaar / bedrock-mongodb

Bedrock mongodb module
Apache License 2.0
2 stars 3 forks source link

Use write concern #65

Closed aljones15 closed 3 years ago

aljones15 commented 3 years ago

Updates our settings so we do not have deprecated uses of w, j, wtimeout, and fsync.

Those options now need to be inside of a writeConcern object.

This addresses:

https://github.com/digitalbazaar/bedrock-mongodb/issues/64

This is a draft PR until it can be confirmed it removes this deprecation error:

Top-level use of w, wtimeout, j, and fsync is deprecated. Use writeConcern instead.

in multiple projects.

Removes deprecation warning in:

  1. bedrock-mongodb
  2. bedrock-edv-storage
  3. bedrock-account
codecov-io commented 3 years ago

Codecov Report

Merging #65 (adce28f) into master (7f5767f) will increase coverage by 0.13%. The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
+ Coverage   59.93%   60.06%   +0.13%     
==========================================
  Files           4        4              
  Lines         292      293       +1     
==========================================
+ Hits          175      176       +1     
  Misses        117      117
Impacted Files Coverage Δ
lib/config.js 100.00% <ø> (ø)
lib/test.config.js 100.00% <ø> (ø)
lib/index.js 56.34% <85.71%> (+0.32%) :arrow_up:

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 7f5767f...adce28f. Read the comment docs.