-
Notifications
You must be signed in to change notification settings - Fork 23
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
Polish search API, frontend and backend #28
Conversation
"version": "0.2.0", | ||
"configurations": [ | ||
{ | ||
"name": "Python Debugger: FastAPI", |
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.
I do like checking in launch scripts in case people need help setting a debuger
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.
Looking really good. It's there!! I just left a few style / naming comments for you to consider. Really well done.
@@ -1,11 +1,12 @@ | |||
|
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.
Is the rest of this Readme up to date with instructions to run the service?
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.
it is now.
@@ -2,13 +2,18 @@ version: '3.9' | |||
|
|||
services: | |||
backend: | |||
image: ghcr.io/redisventures/redis-arxiv-search:latest | |||
# image: ghcr.io/redisventures/redis-arxiv-search:latest |
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.
@tylerhutcherson I think the local should actually build from src instead of pulling but we can talk about this
No description provided.