-
Notifications
You must be signed in to change notification settings - Fork 4
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
SerpApi GET Method Implementation #113
Conversation
A different key will be used in deployment.
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.
Good job, I think it is a great start for the implementation of our web application.
All reviews are completed. I am merging this PR. |
# Create your tests here. | ||
class google_scholar_test_cases(TestCase): |
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.
Hey, I was just looking at this code again and I noticed that the class naming doesn't quite align with our decision to use PEP-8 Style, see class naming convention. Just a friendly reminder for future reference!
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.
Thx for the heads up. I will fix them in a future PR.
This PR includes the implementation of the GET method that utilizes the SerpApi. It also includes the yaml file of Github Actions for automated testing.
Fixes #112