-
Notifications
You must be signed in to change notification settings - Fork 0
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
Migrate from Mariadb to Postgres #400
base: master
Are you sure you want to change the base?
Conversation
mytlogos
commented
Sep 18, 2022
•
edited
Loading
edited
- Use slonik package for some abstractions
- use more typing - fix most indentifier case problems - replace mysql locate with postgres strpos - convert on duplicate to on conflict - fix some syntax errors - fix missing values for addToc
BREAKING CHANGE: migrate from mariadb to postgresql - changed database schema identifiers to use snake_case - removed some unused tables - resetted database version - resetted database migrations - rewrote database api to use slonik - removed deprecated databaseValidator - removed unused contextHelper - removed unnecessary subContext.ts - removed deprecated process server api
Coverage report❌ An unexpected error occurred. For more details, check console
Show new covered files 🐣
Test suite run failedFailed tests: 35/124. Failed suites: 3/5.
Report generated by 🧪jest coverage report action from 70bfbf7 |
- fix import loop of querycontext - use a new(old) way to declare db schemata - fix identifier escaping - provide helper methods for sql.join
- change column to boolean in scraper_hook
SonarCloud Quality Gate failed. 0 Bugs No Coverage information |
SonarCloud Quality Gate failed. 0 Bugs No Coverage information |