-
Notifications
You must be signed in to change notification settings - Fork 53
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
[Refactor] Remove TinyDB #468
Conversation
9a1b348
to
5b60477
Compare
5b60477
to
26b3707
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 🚀 :)
Did you run some Actions to tests :
- submission
- update badges/stars
- Batch | merge check files and push to db
Just to be sure for any inputs/outputs ;)
I believe it's working: |
Cool :) The commit of the PR generated by the submission wf : frankharkins@426423b should have only one file modified, not every member :o |
…into fh-refactor-tinydb
EDIT: After a
This seems to fix the problem (see frankharkins#7). |
Indeed seems to be working as usual now :) About the toml reordering, if you run the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't forget to reverse the changes applied in .toml
files
97df995
to
6fa7f51
Compare
38d0f25
to
ee8f97c
Compare
Tested |
Summary
Removes TinyDB in favour of reading/writing from TOML files directly.