fix(sqlite): default [development]
URL to :memory:
#926
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
currently gives you
There are a few ways to solve this but I think
@cap-js/sqlite
should determine itself that the default URL for[development]
is:memory:
, and not rely on the default defined in@sap/cds
(see the_databases
section incds-requires.js
), which it currently does.This is a non-breaking change, as today the
cds
-providedurl: ':memory:'
wins, so even before we deployed to an in-memory DB by default. This just fixes a confusing log and prepares for the next major version, where we can then remove the redundant and outdatedcredentials.database
.