Skip to content
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.

Sign up for GitHub

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

Jump to bottom

55 switch to sqlite #81

Merged
merged 7 commits into from
Dec 8, 2016
Merged

55 switch to sqlite #81

merged 7 commits into from
Dec 8, 2016

Conversation

leviable
Copy link
Member

@leviable leviable commented Dec 8, 2016

levi-rs added 6 commits December 6, 2016 13:26
Sqlite3 working
b225fe5
Completely switched over to SQLite
629b3b8 
 - Create SQLite database and memes table if they don't exist
 - Convert previous MySQL functionality to new sqlite db
 - Use reddit_id for primary key instead of auto incrementing an int
 - Add last_seen column
 - Update last_seen if a meme has been seen previously
Fix unit tests for memes.py
677cf81
Make testing more DRY
2a9f278 
 - Move fixtures to conftest
 - Move string definitions to __init__
Unit tests back to 100%
479584f 
 - Unit tests back to 100%
 - Moved most fixtures to conftest.py
 - General code cleanup and readability improvements
Update sample INI and README
5de6f6c
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling fc260f0 on 55-switch-to-sqlite into b93264e on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 42b28d2 on 55-switch-to-sqlite into b93264e on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling b2aa7fc on 55-switch-to-sqlite into b93264e on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 4e3690c on 55-switch-to-sqlite into b93264e on master.

@leviable leviable merged commit 3da6f9c into master Dec 8, 2016
@leviable leviable deleted the 55-switch-to-sqlite branch December 8, 2016 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@leviable @coveralls