knex / knex-schema-inspector

Utility for extracting information about existing DB schema
MIT License
99 stars 43 forks source link

fix: postgres schema #120

Open pavansandeep2910 opened 2 years ago

pavansandeep2910 commented 2 years ago

fixed issue for schema name with space #119

rijkvanzanten commented 2 years ago

Mind adding an extra test case for this as well? Just to make sure it doesn't break in the future

pavansandeep2910 commented 2 years ago

Mind adding an extra test case for this as well? Just to make sure it doesn't break in the future

do I need to add an extra test case for all the inspector functions or any one function?

rijkvanzanten commented 2 years ago

Technically speaking, for full coverage we should do one for each of the methods yeah