Conversation
|
Thanks for the PR! I haven't had the time to take a look at this yet, but I haven't forgotten about it |
|
As further context, before I have to search for it again in Slack; This PR was first opened as #931 Suggestions from Slack, not very relevant- A suggestion was made there to 'make the output a little prettier' - There the suggestion was made to log it line by line instead of logging the object itself.
The tests that should be added here should use simplified versions of the test config and check the output is as expected. Suggested tests are;
These tests can be based on https://github.com/sequelize/cli/blob/main/test/version.test.js @reilg do you think you can work on this further with this information? |
Pull Request check-list
Please make sure to review and check all of these items:
npm run testpass with this change (including linting)?Description of change
Adds the
db:infocommand which displays the config information.