Skip to content

Conversation

@Kilo59
Copy link
Owner

@Kilo59 Kilo59 commented May 9, 2020

  • merge location package into a single module
  • use pydantic for Location serialization
  • fix redis caching for csbs
  • fix redis caching for nyt

@Kilo59 Kilo59 force-pushed the redis-serialization branch 3 times, most recently from a737390 to bb0b2d5 Compare May 9, 2020 23:15
@Kilo59 Kilo59 self-assigned this May 9, 2020
@Kilo59 Kilo59 force-pushed the redis-serialization branch from bb0b2d5 to e312355 Compare May 9, 2020 23:18
@Kilo59 Kilo59 force-pushed the redis-serialization branch from e312355 to 838951d Compare May 9, 2020 23:21
@Kilo59 Kilo59 marked this pull request as draft May 10, 2020 01:17
@Kilo59 Kilo59 requested a review from tanwinn May 10, 2020 01:17
@tanwinn tanwinn marked this pull request as ready for review May 20, 2020 01:07
Copy link
Collaborator

@tanwinn tanwinn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to download the python-dev package because pf psutils though I didn't have to before. I also didn't see that dependency in Pipfile. Do you know anything about it? Pretty sure it wasn't a coincidence that you mentioned python-dev today 😅

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.

3 participants