Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add support for the paranoid flag #1512

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SteGun93
Copy link

@SteGun93 SteGun93 commented Jul 4, 2024

Pull Request check-list

Please make sure to review and check all of these items:

  • Does npm run test pass with this change (including linting)?
  • Does the description below contain a link to an existing issue (Closes Config option are not being picked up and used. #1139) or a description of the issue you are solving?
  • Have you added new tests to prevent regressions?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Description of change

The following files have been changed:

model_generate.js
yargs.js
model.js
model-helper.js
migration-helper.js
create-table.js
create.test.js

solved issue:

#1139

The following test has been adapted:

model/create.test.js

update to the documentation:

In order to be able to assign the paranoid flag via the CLI, it is possible to set the flag --paranoid true after the model:generate

You can also set the paranoid flag to true in the .sequelizerc , so all created models and migrations will be paranoid by default.

If you have set paranoid to true in the config, you can set paranoid to false for individual tables by setting the flag to --paranoid false via the CLI after the model:generate

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Config option are not being picked up and used.
1 participant