Miroslavsckaya
/
tg_rss_bot
1
1
Fork 0
Code Issues 6 Pull Requests Projects 1 Releases Activity

Database migration #45

Merged
Miroslavsckaya merged 8 commits from database_migration into master 2022-08-21 20:43:36 +00:00
Conversation 0 Commits 8 Files Changed 3 +42 -22
Miroslavsckaya commented 2022-08-21 13:06:47 +00:00
Owner
Copy Link

Closes #16

Closes #16
Miroslavsckaya added 1 commit 2022-08-21 13:06:48 +00:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is failing Details
8766e36aa6 add database migrations
Miroslavsckaya added 1 commit 2022-08-21 16:17:55 +00:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is failing Details
8d4aa91d6f code style changes
Miroslavsckaya changed title from Database migration to WIP:Database migration 2022-08-21 16:18:26 +00:00
Miroslavsckaya added 1 commit 2022-08-21 16:20:39 +00:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
dfa0a137ba little code style changes
Miroslavsckaya added 1 commit 2022-08-21 16:28:46 +00:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
5aa544cc9c delete IF NOT EXISTS in create table sql queries
skobkin approved these changes 2022-08-21 16:50:05 +00:00
skobkin left a comment
Collaborator
Copy Link

Looks good, but here's a few small points of optional improvement.

Looks good, but here's a few small points of **optional** improvement.
database.py Outdated
@ -2,6 +2,7 @@ from logging import Logger
import psycopg2
from psycopg2.extensions import connection
from psycopg2.extras import DictCursor, DictRow
from yoyo import read_migrations, get_backend
skobkin commented 2022-08-21 16:35:08 +00:00
Collaborator
Copy Link

Ascending sort maybe?

Ascending sort maybe?
skobkin marked this conversation as resolved
database.py
@ -176,3 +160,1 @@
' guid TEXT'
')'
)
def __migrate(self, dsn: str) -> None:
skobkin commented 2022-08-21 16:35:52 +00:00
Collaborator
Copy Link

Catch and log some errors maybe? 🙂

Catch and log some errors maybe? 🙂
skobkin marked this conversation as resolved
migrations/0000.initial_schema.py Outdated
@ -0,0 +1,20 @@
from yoyo import step
steps = [
step('CREATE TABLE users (id SERIAL PRIMARY KEY, telegram_id INTEGER NOT NULL UNIQUE)'),
skobkin commented 2022-08-21 16:49:31 +00:00
Collaborator
Copy Link

You can add indentation for each table for readability.

You can add indentation for each table for readability.
skobkin marked this conversation as resolved
Miroslavsckaya added 1 commit 2022-08-21 20:03:55 +00:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is failing Details
90c93528ec add indentation in sql queries
Miroslavsckaya added 1 commit 2022-08-21 20:15:10 +00:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is failing Details
d3d33bb718 add docstring and logging to Database.__migrate method
Miroslavsckaya added 1 commit 2022-08-21 20:25:20 +00:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is failing Details
52023fcd1a correct docstrings in database.py
skobkin approved these changes 2022-08-21 20:35:50 +00:00
skobkin left a comment
Collaborator
Copy Link

Let's merge it 😄

Let's merge it 😄
Miroslavsckaya added 1 commit 2022-08-21 20:39:11 +00:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
fe2a9cc515 code style changes
Miroslavsckaya changed title from WIP:Database migration to Database migration 2022-08-21 20:42:04 +00:00
Miroslavsckaya merged commit 31c112a4fa into master 2022-08-21 20:43:36 +00:00
Miroslavsckaya deleted branch database_migration 2022-08-21 20:43:36 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
skobkin
Labels
Clear labels   
bug

Something is not working

  
enhancement

New feature

  
epic

   
invalid

Nothing to fix or wrong report

  
new feature

   
refactoring

   
research needed

   
wontfix

No need to resolve

No Label
bug
enhancement
epic
invalid
new feature
refactoring
research needed
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Miroslavsckaya/tg_rss_bot#45
No description provided.
Delete Branch "database_migration"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?