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

Document .sequelizerc configuration options #11553

Merged
merged 3 commits into from Oct 18, 2019

Conversation

samuliasmala
Copy link
Contributor

The .sequelizerc has several configuration options which are defined and described in the source code (cli/src/core/yargs.js) but not included in the documentation.

The .sequelizerc has several configuration options which are defined and described in the source code (cli/src/core/yargs.js) but not included in the documentation.
@sushantdhiman sushantdhiman added the status: awaiting response For issues and PRs. OP must respond (or change something, if it is a PR). Maintainers have no action label Oct 16, 2019
@sushantdhiman
Copy link
Contributor

Linting fails

@papb papb added the type: docs For issues and PRs. Things related to documentation, such as changes in the manuals / API reference. label Oct 16, 2019
@codecov
Copy link

codecov bot commented Oct 17, 2019

Codecov Report

Merging #11553 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #11553   +/-   ##
=======================================
  Coverage   96.27%   96.27%           
=======================================
  Files          94       94           
  Lines        9184     9184           
=======================================
  Hits         8842     8842           
  Misses        342      342

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 3517eb7...84ec101. Read the comment docs.

Add blank line to fix MD032 - Lists should be surrounded by blank lines
@sushantdhiman sushantdhiman merged commit 7d251bd into sequelize:master Oct 18, 2019
@sushantdhiman
Copy link
Contributor

🎉 This PR is included in version 5.20.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released status: awaiting response For issues and PRs. OP must respond (or change something, if it is a PR). Maintainers have no action type: docs For issues and PRs. Things related to documentation, such as changes in the manuals / API reference.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants