-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add load_or_new
function that loads a DB if available, and if not, create DB
#22
base: master
Are you sure you want to change the base?
Conversation
…es a new database.
Thanks for your contribution @sigaloid ! Can you please fix the clippy errors before we merge your PR? Also - it'd be great if you can add a test. |
Hi, I think I've fixed the clippy error. Also, I'm unfamiliar with the test structure so a test that I write would likely not help very much or be poorly written 😔 |
@sigaloid I'm sorry for the late response. I think you can add a simple test here: Also - the clippy errors were fixed here: #25 so you can update your branch and it should work now |
new_or_load
function that loads a DB if available, and if not, create DBload_or_new
function that loads a DB if available, and if not, create DB
Successfully added tests, looks like it's passing in my repo but needs a maintainer to run here. |
@sigaloid please see a few clippy and fmt issues |
👀 |
No description provided.