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.

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

Change database infrastructure #62

Merged
merged 18 commits into from
Jan 1, 2021
Merged

Conversation

pylipp
Copy link
Owner

@pylipp pylipp commented Dec 28, 2020

No description provided.

- prepare for database restructuring (database shall span more than a
  year)
- use term suitable for financing context
- thanks tpope for vim-abolish!
- followed
  https://tinydb.readthedocs.io/en/latest/changelog.html#v4-0-0-2020-05-02
- 'Element' is deprecated (msiemens/tinydb#158); 'Document' is
  constructed differently, and has different attribtues
- apparently empty tables are not written to disk anymore
- clean up in JsonTinyDbPocketTestCase
@coveralls
Copy link

coveralls commented Dec 28, 2020

Coverage Status

Coverage decreased (-0.3%) to 99.528% when pulling 0868e1f on change-database-infrastructure into bd1a112 on master.

@pylipp pylipp force-pushed the change-database-infrastructure branch from 6a5e921 to 16e1ef2 Compare December 29, 2020 17:11
@pylipp pylipp force-pushed the change-database-infrastructure branch from 16e1ef2 to e9f250a Compare December 30, 2020 22:18
Irrelevant since using powerful date parsing provided by python-dateutil
- update that requires the convert-periods-to-pocket functionality
- start and end of recurrent entries may not lie in current year
Since a Pocket does not necessarily map to a single year anymore, this
method is redundant
For 'infinitely' recurring entries; previously recurrent entries ended
on 12-31 if no end date specified
@pylipp pylipp force-pushed the change-database-infrastructure branch from f06471f to b7821da Compare January 1, 2021 17:59
@pylipp pylipp marked this pull request as ready for review January 1, 2021 18:21
@pylipp pylipp merged commit bd08985 into master Jan 1, 2021
@pylipp pylipp deleted the change-database-infrastructure branch January 1, 2021 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants