ever-co / ever-demand

Everยฎ Demandโ„ข - Open Commerce Platform - https://everdemand.co
https://everdemand.co
GNU Affero General Public License v3.0
1.73k stars 462 forks source link

An in-range update of mongoose is breaking the build ๐Ÿšจ #781

Closed greenkeeper[bot] closed 4 years ago

greenkeeper[bot] commented 5 years ago

The dependency mongoose was updated from 5.6.7 to 5.6.8.

๐Ÿšจ View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

mongoose is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Status Details - โŒ **ci/circleci: build-backend-api:** Your tests are queued behind your running builds ([Details](https://circleci.com/gh/ever-co/ever/18016?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link)). - โŒ **ci/circleci: build-monorepo-root:** Your tests are queued behind your running builds ([Details](https://circleci.com/gh/ever-co/ever/18018?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link)). - โŒ **ci/circleci: build-carrier-mobile-ionic:** CircleCI is running your tests ([Details](https://circleci.com/gh/ever-co/ever/18014?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link)). - โŒ **ci/circleci: build-admin-website-angular:** CircleCI is running your tests ([Details](https://circleci.com/gh/ever-co/ever/18012?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link)). - โœ… **ci/circleci: build-shop-website-angular:** Your tests passed on CircleCI! ([Details](https://circleci.com/gh/ever-co/ever/18013?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link)). - โŒ **ci/circleci: build-shop-mobile-ionic:** CircleCI is running your tests ([Details](https://circleci.com/gh/ever-co/ever/18017?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link)). - โŒ **ci/circleci: build-merchant-tablet-ionic:** Your tests failed on CircleCI ([Details](https://circleci.com/gh/ever-co/ever/18015?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link)).

Commits

The new version differs by 22 commits.

  • bd03c4d chore: release 5.6.8
  • 6fd4975 chore: add ehic cards as opencollective sponsor
  • 01b182e refactor: handle arrays underneath mixed paths for #6405
  • 4b2f770 Merge pull request #8033 from Mangosteen-Yang/new-feature-#6403
  • f58c45f fix(document): allow saving an unchanged document if required populated path is null
  • c698590 test(document): repro #8018
  • eced064 docs(transactions): clarify relationship between session in docs and MongoDB driver ClientSession class, link to driver docs
  • c8e270e docs(plugins): rewrite plugins docs to be more modern and not use strange = exports syntax
  • f4aaa9c test: fix tests
  • 9db4857 fix(query): make findOneAndReplace() work with orFail()
  • f10f621 test(query): repro #8012
  • 1670ff1 Merge branch 'master' of github.com:Automattic/mongoose
  • 710935d fix(aggregate): allow modifying pipeline in pre('aggregate') hooks
  • a2584a1 test(aggregate): repro #8017
  • e4169a2 debug support {color: false}

There are 22 commits in total.

See the full diff

FAQ and help There is a collection of [frequently asked questions](https://greenkeeper.io/faq.html). If those donโ€™t help, you can always [ask the humans behind Greenkeeper](https://github.com/greenkeeperio/greenkeeper/issues/new).

Your Greenkeeper Bot :palm_tree:

greenkeeper[bot] commented 5 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 19 commits.

  • 188b4e7 chore: release 5.6.9
  • 347eeeb chore: add papersowl as sponsor
  • 379a807 style: fix lint
  • 4a84546 fix(schema): fix test failures from previous fix of #8034
  • 6149ba3 docs(index): add dcsl as sponsor
  • f47e936 fix(schema): allow declaring ObjectId array with { type: 'ObjectID' }, last 'D' case insensitive
  • bc71f25 test(schema): repro #8034
  • ad41e38 Merge branch 'master' of github.com:Automattic/mongoose
  • 9023f30 docs(guide): add missing further reading link and link to other tutorial
  • 419cfef Merge pull request #8048 from Fonger/fix/version-error-memory-leak
  • 612c4f9 chore: now working on 5.6.9
  • 4408e55 fix(cursor): correctly handle batchSize option with query cursor
  • eff168d test(query): repro #8039
  • b850278 fix(populate): handle virtual populate with count = 0 if virtual embedded in doc array
  • 548ab98 test(populate): repro #7573 part 2

There are 19 commits in total.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 44 commits.

  • 3847007 chore: release 5.6.10
  • 3feb60b docs(query): add findByIdAndReplace() to docs
  • 5c2fbc5 test: fix tests re: #8073
  • 20fa775 refactor: remove async.parallel() calls
  • 5595ead chore: add ghostinspector as sponsor from opencollective
  • cbdadf2 Merge pull request #8070 from rich-earth/master
  • d0fba5e fix: fix tests re: #8062
  • aa5e737 fix(document): skip casting when initing a populated path
  • 5f416bf test(document): repro #8062
  • e98d2f7 refactor(document): remove async.each() dependencies
  • 7c7b744 Merge branch 'master' of github.com:Automattic/mongoose
  • 6eeb48e fix(document): prevent double-calling validators on mixed objects with nested properties
  • f023247 test(document): repro #8067
  • 69716ac Merge pull request #8066 from Mangosteen-Yang/master
  • 813a8d2 Update handleImmutable.js

There are 44 commits in total.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 56 commits.

  • 64825fa chore: release 5.6.11
  • c8aaa01 fix(document): make validateUpdatedOnly option handle pre-existing errors
  • b0fd1b0 docs(query): add missing options to setOptions()
  • b81a62a style: fix lint
  • 314317e test: fix tests re: #8075
  • 57721e4 test: alternative test re: #8075
  • 58766d9 fix(model): allow passing options to exists()
  • 1f299de docs(middleware): clarify that you must add middleware before compiling your model
  • 264a1b4 fix: bump kareem -> 2.3.1 to throw readable error if middleware callback isnt a function
  • 5996613 chore: now working on 5.6.11
  • a6d1e7a Merge pull request #8089 from felipeplets/patch-1
  • ad0666b Fix issue when using find
  • 3847007 chore: release 5.6.10
  • 3feb60b docs(query): add findByIdAndReplace() to docs
  • 5c2fbc5 test: fix tests re: #8073

There are 56 commits in total.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 74 commits.

  • 70f678e chore: release 5.6.12
  • c97c83f style: fix lint
  • da05549 test(schema): add test re: #8124
  • 8625bb9 Merge pull request #8124 from StphnDamon/hotfix/schematype-cloning
  • 9b8b7b3 Add missing getters and setters copy when cloning (see issue #8111)
  • 03dfc6a refactor(model): use internal parallelLimit() instead of async
  • 412e387 docs(plugins): fix link to npm keywords doc
  • dba0336 fix(schema): improve fix for #8111
  • 50aeda0 fix(schema): handle required validator correctly with clone()
  • 29b232e test(schema): repro #8111
  • 970faf4 fix(discriminator): avoid unnecessarily cloning schema to avoid leaking memory on repeated discriminator() calls
  • 86e1ab5 chore: remove lockfile
  • 6a9297c Merge pull request #8112 from TheTrueRandom/master
  • 85521cb docs(schematypes): add missing 'the'
  • 249f4c6 Merge pull request #8109 from jgcmarins/patch-1

There are 74 commits in total.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 86 commits.

  • 5b11924 chore: release 5.6.13
  • 3a105e6 test(model): add test for #8130
  • cad316a Merge pull request #8130 from sibelius/patch-1
  • 6a9abe8 fix(document): ensure nested mixed validator gets called exactly once
  • a57e491 test(document): repro #8117
  • 7879913 fix(parallel): fix parallelLimit when fns is empty
  • 5e289d7 style: fix lint
  • 946f7da chore: now working on 5.6.13
  • 006a0f6 Merge pull request #8125 from gradecam/feature/fix_justOne_default
  • 763e3e4 Fix for bug which doesn't properly honor defaults for justOne
  • b92003b Add unit test to demonstrate justOne issue
  • 5b26e52 Created feature branch fix_justOne_default
  • 70f678e chore: release 5.6.12
  • c97c83f style: fix lint
  • da05549 test(schema): add test re: #8124

There are 86 commits in total.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 136 commits.

  • a4c3e10 chore: release 5.7.0
  • e2434ca chore: add moneypug and vpngorilla as opencollective sponsors
  • fcf035e Merge pull request #8145 from Automattic/5.7
  • e6e3314 fix: be more rigorous about only montoring pool events if useUnifiedTopology is set re: #8083
  • dc23ed1 docs: add useUnifiedTopology to docs re: #8083
  • 661f102 fix(connection): handle reconnect, reconnectFailed, timeout events with useUnifiedTopology
  • bc616c8 fix(connection): work around NODE-2155 re: #8083
  • 9041fd4 fix: add useUnifiedTopology option analagous to useNewUrlParser
  • 037fbe5 refactor(populate): assign using all separate models after all queries are done re: #3812
  • bc3daf8 chore: remove renovate, unused currently
  • 6dd8816 Merge pull request #8138 from joebowbeer/patch-1
  • c4e0e99 Update guide.pug
  • eb75b54 refactor(populate): only call assign() after all model queries for a path are called re: #3812
  • 39566f1 refactor(populate): move execPopulateQuery() calls after all queries are computed re: #3812
  • af14c61 refactor(populate): move execQuery() into separate function

There are 136 commits in total.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 152 commits.

  • 32c4e3c chore: release 5.7.1
  • 0aec7ad fix(document): fix test failures from fix for #8149
  • b4d0303 Merge branch 'master' of github.com:Automattic/mongoose
  • 8d86802 fix(document): throw strict mode error if setting an immutable path with strict mode: false
  • 74f558c test(document): repro #8149
  • 3f9c9b8 Merge pull request #8151 from fernandolguevara/patch-2
  • 6e86500 fix(model): make syncIndexes() handle changes in index key order
  • ce9d8d0 test(model): repro #8135
  • b8cf292 chore: remove paperleaf and papersowl as sponsors
  • 691aabb fix(mongoose): support passing options object to Mongoose constructor
  • 293319b chore: now working on 5.7.1
  • 2a4296b docs(connection+mongoose): add useUnifiedTopology option to connect() and openUri() docs
  • c8038df style: fix lint
  • 6b28ee0 Merge pull request #8148 from ouyuran/patch-1
  • 483c90a fix findOneAndUpdate with { runValidators: true } options

There are 152 commits in total.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 188 commits.

  • 43b63ae chore: release 5.7.3
  • 06112b0 docs(validation): remove deprecated isAsync from validation docs in favor of emphasizing promises
  • 7fee719 docs(documents): add overwriting section
  • 98b5a73 fix: make CoreMongooseArray#includes() handle fromIndex parameter
  • 6c91dea style: fix lint
  • 9bb4b03 refactor: remove async as a prod dependency
  • 3647292 refactor(cursor): remove async.queue() from eachAsync() re: #8073 #5502
  • e60db1b refactor(cursor): remove dependency on async.times()
  • c5b2355 docs(promises): add note about queries being thenable
  • da77b8d Merge pull request #8192 from birdofpreyru/fix-8093-1
  • c371500 fix(update): cast right hand side of $pull as a query instead of an update for document arrays
  • 9d455ad test(update): repro #8166
  • 8c98a3a chore: now working on 5.7.3
  • 0a33412 fix(populate): handle virtual populate of an embedded discriminator nested path
  • b42d0f5 test(populate): repro #8173 #6488

There are 188 commits in total.

See the full diff

greenkeeper[bot] commented 4 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 18 commits.

  • ede5aef chore: release 5.7.4
  • 402db1a fix(model): support passing options to Model.remove()
  • 7a20276 fix(schema): handle required: null and required: undefined as required: false
  • 9b4a323 test(schema): repro #8219
  • fb66f3a fix: use options constructor class for all schematypes
  • d5e0c73 refactor: use options constructor for array, buffer, date schematypes re: #8012
  • 2d0955d refactor: use SchemaNumberOptions class for schematype number re: #8012
  • 166cd88 refactor: use SchemaStringOptions class for string schematype options re: #8012
  • 523c181 fix(update): support updating array embedded discriminator props if discriminator key in $elemMatch
  • f277d94 test(update): repro #8063
  • d19b849 test: add test coverage for parallelLimit()
  • 294c191 refactor(schematype): add SchemaTypeOptions class re: #8012
  • dc0025b fix(populate): allow accessing populate virtual prop underneath array when virtual defined on top level
  • f8db7ce test(populate): repro #8198
  • 9a1d494 style: fix lint

There are 18 commits in total.

See the full diff

greenkeeper[bot] commented 4 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 38 commits.

  • 40a879b chore: release 5.7.5
  • 159457d chore: add vpn black friday as sponsor
  • e6285ea Merge pull request #8244 from AbdelrahmanHafez/master
  • d9163f5 fix: correct order for declaration
  • cec9dda Minor refactor to ValidationError
  • 13ae085 docs(index): add favicon to home page
  • 96ce0eb style: fix lint
  • 973b1e0 docs: add schema options to API docs
  • cdfb507 chore: add useUnifiedTopology for tests re: #8212
  • 936ddfb fix(update): handle subdocument pre('validate') errors in update validation
  • 98b3b09 test(update): repro #7187
  • b9c1012 docs(middleware): add note about accessing the document being updated in pre('findOneAndUpdate')
  • 327b47a fix(subdocument): make subdocument#isModified use parent document's isModified
  • 54db026 test(subdocument): repro #8223
  • 89eb449 chore: now working on 5.7.5

There are 38 commits in total.

See the full diff

greenkeeper[bot] commented 4 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 75 commits.

  • 28e8ac4 chore: release 5.7.7
  • f4bd463 fix(populate): make populate virtual consistently an empty array if local field is only empty arrays
  • ca4f52b test(populate): repro #8230
  • ecd8242 docs(query): make note that filter param to find() can be an ObjectId re: #8268
  • b03faf5 fix(query): allow findOne(objectid) and find(objectid)
  • d7ceb89 test(query): repro #8268
  • 068e57a chore: now working on 5.7.7
  • 5656b4e chore: release 5.7.6
  • fab4f5d test(schema): fix tests on node v4 and v5
  • 29c5f1a fix(options): add missing minlength and maxlength to SchemaStringOptions
  • a81211d fix(populate): add document array subpaths to parent doc populated() when calling DocumentArray#push()
  • 4e900eb test(populate): repro #8247
  • d69ab22 fix(document): support calling Document constructor directly in Node.js
  • 9b986f5 test(document): repro #8237
  • e5f8875 chore: add bonus.ca to opencollective sponsors

There are 75 commits in total.

See the full diff

greenkeeper[bot] commented 4 years ago

Your tests for group default are passing again with this update. Explicitly upgrade default to this version ๐Ÿš€

Commits

The new version differs by 93 commits.

  • 8c79cdd chore: clean up unnecessary eslint rules that are confusing docs parsing
  • fcbed7b chore: release 5.7.8
  • b3c95de docs: add links to clarify truthy/falsy
  • 2694cbd docs: document return types for .discriminator()
  • 4c090c5 fix(cursor): throw error when using aggregation cursor as async iterator
  • 69bf575 docs(aggregate): make aggregation Symbol.iterator docs actually use aggregate()
  • c1e49c3 test: fix tests re: #8273
  • dfde779 fix(document): allow manually populating path within document array
  • b86749e fix(document): make Document class an event emitter to support defining documents without models in node
  • 374246d docs(connection): add note about exporting schemas, not models, in multi connection paradigm
  • 5b6eeab docs: clarify that transforms defined in toObject() options are applied to subdocs
  • 7dd63fd fix(schema): retain _id: false in schema after nesting in another schema
  • 78d5c4b test: repro #8274
  • 0d057fe docs(model): remove unnecessary *
  • 7dbf96c refactor(schema): create separate schematype for DocumentArrayPaths instead of ad-hoc within getPositionalPath()

There are 93 commits in total.

See the full diff

greenkeeper[bot] commented 4 years ago

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 107 commits.

  • 537217e chore: release 5.7.9
  • 97d6900 test: fix tests for node 4.x and 5.x
  • 03ef4d8 fix(document): support getter setting virtual on manually populated doc when calling toJSON()
  • d4648ee test(document): repro #8295
  • 6c1d46a fix(model): allow objects with toBSON() to make it to save()
  • de3eee5 docs: add link to custom casting guide
  • 1c4d994 chore: now working on 5.7.9
  • 9b0e96d fix(populate): make retainNullValues set array element to null if foreign doc with that id was not found
  • f0aeddc test(populate): repro #8293
  • d02ebd2 fix(schema): support setting schema path to an instance of SchemaTypeOptions
  • 081f2ec test(schema): repro #8292
  • 328f94e chore: remove opencollective sponsor
  • 4f2fe4f chore: add new opencollective sponsors
  • 0bdc11e chore: add crosswordsolver as a sponsor
  • 8c79cdd chore: clean up unnecessary eslint rules that are confusing docs parsing

There are 107 commits in total.

See the full diff

greenkeeper[bot] commented 4 years ago

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 11 commits.

  • 95b25ce chore: release 5.7.10
  • 2753e1c chore: fix package.json config for #8322
  • 284b44d Merge pull request #8322 from stieg/improve-tests
  • 5c04b5d Merge pull request #8321 from dandv/patch-1
  • 9e31710 build(test) Improve test target
  • 689791b Illustrate Promise API, move Import after Install
  • e9c69b3 perf(cursor): remove unnecessary setTimeout() in eachAsync(), 4x speedup in basic benchmarks
  • bea0b6d chore: ensure all node tests do require('./common') first so they can work independently
  • 6067dcd chore: alternative crosswordsolver logo re: #8301
  • e6859bc test: clean up test for timestamps with insertMany() re: #8304
  • 81c61f6 chore: add opencollective sponsors

See the full diff

greenkeeper[bot] commented 4 years ago

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 11 commits.

  • 9970a6f chore: release 5.7.11
  • e268572 Merge pull request #8341 from Fonger/fix/memory-leak
  • e5c6ded test(connection): fix tests re: #8337
  • 0c0833a test(model): add test for issue #8040 re: #8048 #8326
  • 4ec852d fix(connection): bubble up connected/disconnected events with unified topology
  • 98facde Merge pull request #8332 from AbdelrahmanHafez/master
  • c900645 Merge pull request #8326 from Fonger/patch-5
  • 91ea10e fix: upgrade to mongodb driver 3.3.4
  • a91dc93 Add test for #8331
  • 6c1dcb2 Fixes #8331
  • de583bb fix(model): delete $versionError after saving to prevent memory leak

See the full diff