egg-sequelize
egg-sequelize copied to clipboard
feat: add Sequelize to config's declaration and test it works with sequelize-typescript
Checklist
- [x]
npm testpasses - [x] tests and/or benchmarks are included
- [x] documentation is changed or added
- [x] commit message follows commit guidelines
Affected core subsystem(s)
Description of change
index.d.ts增加Sequelize字段以反映真实的设置,以及支持TypeScript里定制这个字段。README.md增加如何在TypeScript版 egg 项目里使用sequelize-typescript的说明。test目录下增加在TypeScript版 egg 项目里使用sequelize-typescript的测试用例。README.md增加如何在本地运行此项目的单元测试的说明。- 增加
test.sh以方便本地单元测试。
Codecov Report
Merging #83 (8a3c1f4) into master (f585da5) will decrease coverage by
1.33%. The diff coverage is100.00%.
@@ Coverage Diff @@
## master #83 +/- ##
==========================================
- Coverage 85.71% 84.37% -1.34%
==========================================
Files 4 4
Lines 63 64 +1
==========================================
Hits 54 54
- Misses 9 10 +1
| Impacted Files | Coverage Δ | |
|---|---|---|
| lib/loader.js | 83.05% <100.00%> (-1.44%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update a126c2b...a9f7d6e. Read the comment docs.
第一个提交测试通过,第二个提交没有本质改动,测试败在 npminstall 这一步。我在 fork 的 TravisCI 也重现了,清除 Cache 后,通过了。
@dead-horse @fengmk2 Any chance to get it approved and merged?