hackycy / egg-typeorm

🔥 TypeORM plugin for Egg.js
MIT License
13 stars 5 forks source link

添加支持多种格式的 ormconfig #3

Closed zellzh closed 4 years ago

zellzh commented 4 years ago
Checklist
Affected core subsystem(s)

function loadOrmConfig

Description of change

将多种格式的 ormconfig 转为 js 对象, 只对 loadOrmConfig 做修改, 不影响其他模块

hackycy commented 4 years ago

原本loadOrmConfig方法只是为了读取一个entitiesDir的配置。 其实和原始配置没有冲突的。你可以照样使用定义文件的配置

例如 ormconfig.yaml 文件

default:
  host: "localhost"
  port: 3306
  username: "test"
  password: "test"
  database: "test"
  entitiesDir: app/entity/db1

db2:
  entitiesDir: app/entity/db2

db3:
  entitiesDir: app/entity/db3

插件运行的时候主要是使用config.typeorm下的配置和ormconfig.yaml对应配置下的entitiesDir字段。

运行的时候主要是一定要配置ormconfig.yaml下的entitiesDir这个字段。

其余像TypeORM官方的类似ormconfig.[format]文件,例如ormconfig.json等等。你重复定义这类文件是没有影响的。

zellzh commented 4 years ago

是这样的 其实我只给加载配置文件函数加了几个判断, 这样 entitiesDir 配置项也可以直接添加给 .js / .json 等其他配置文件 就可以避免必须使用 yaml 文件了, 但是都必须添加 entitiesDir 选项

hackycy commented 4 years ago

这个思路可以的,我再思考下吧,我可以看看能不能一并丢弃掉config.typeorm下定义的配置。

而且代码Eslint没过呀。