Skip to content

feat: Optionally set the schema for table creation#66

Merged
hsluoyz merged 4 commits intoapache:masterfrom
js-lowes:optionally-set-schema
Sep 21, 2022
Merged

feat: Optionally set the schema for table creation#66
hsluoyz merged 4 commits intoapache:masterfrom
js-lowes:optionally-set-schema

Conversation

@js-lowes
Copy link
Contributor

Support to optionally set the schema for table


Why?

Some RDBMS support the idea of schema being a separated namespace under a database, e.g. Postgres.

What was changed?

  1. Propagate the schema option to the table creation exactly how tableName was propagated.
  2. Updated README to include instructions on how to test locally

In postgres the table can be created under different schemas and not
just "public"
@casbin-bot
Copy link

@nodece @Zxilly @Shivansh-yadav13 please review

@CLAassistant
Copy link

CLAassistant commented Sep 19, 2022

CLA assistant check
All committers have signed the CLA.

@hsluoyz
Copy link
Member

hsluoyz commented Sep 19, 2022

@nodece @Zxilly @Shivansh-yadav13 please review

@js-lowes
Copy link
Contributor Author

@hsluoyz had a lint error, I fixed it

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3083837725

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.09%) to 84.795%

Totals Coverage Status
Change from base Build 3030254759: 0.09%
Covered Lines: 113
Relevant Lines: 122

💛 - Coveralls

@Zxilly
Copy link
Contributor

Zxilly commented Sep 21, 2022

lgtm, but please follow semantic commit rules.

@hsluoyz hsluoyz merged commit 3fcc396 into apache:master Sep 21, 2022
Shivansh-yadav13 pushed a commit to Shivansh-yadav13/sequelize-adapter that referenced this pull request Sep 21, 2022
* Optionally set the schema for table creation

In postgres the table can be created under different schemas and not
just "public"

* updated readme

* fix - description

* fix - format
@js-lowes js-lowes deleted the optionally-set-schema branch September 21, 2022 11:57
js-lowes added a commit to js-lowes/sequelize-adapter that referenced this pull request Sep 21, 2022
hsluoyz pushed a commit that referenced this pull request Sep 22, 2022
* feat: Add in schema in previos PR: #66

* fix: Require a frozen lockfile when build

"--frozen-lockfile" will make it so that the yarn build process
is reproducible and uses the versions of dependencies defined in
the "yarn.lock" file.
@github-actions
Copy link

🎉 This PR is included in version 2.4.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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants