-
Notifications
You must be signed in to change notification settings - Fork 55
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
Feature/fundamentals #581
Feature/fundamentals #581
Conversation
Codecov Report
@@ Coverage Diff @@
## master #581 +/- ##
==========================================
+ Coverage 94.73% 95.29% +0.55%
==========================================
Files 5 5
Lines 228 255 +27
==========================================
+ Hits 216 243 +27
Misses 12 12
Continue to review full report at Codecov.
|
cea192b
to
b76f269
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.
@atarax thank you for this! This looks great, I appreciate the detail on the tests + docstrings.
2 questions before merging this:
- Would you like to be added to the Contributors.rst?
- Would you be interested in adding a section to the README.rst showing people basic usage of these new methods you've added? If not , we can add an issue for someone else to add examples.
b76f269
to
73d92e5
Compare
|
@atarax This looks great, thanks for adding the examples to the Readme! I don't see your edits on |
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.
One small doc change and this will be good to go.
README.rst
Outdated
# it was reported to SEC. Set to False if you want to get data containing | ||
# corrections | ||
fundamentals_stmnts = CLIENT.get_fundamentals_statements('GOOGL', | ||
startDate='2020-1-1', |
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.
Can we edit this the startdate month/date for startDate
are 0
padded like in the earlier example? Not sure if the API will handle the different number of digits correctly, so probably good to err on the safe side.
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 actually can, but i changed it for consistency ;)
73d92e5
to
2fa5f0c
Compare
i don't see a file with that name? oO |
Apologies @atarax, too much context switching between projects lately: here
it is :)
https://github.com/hydrosquall/tiingo-python/blob/master/AUTHORS.rst
…On Wed, Feb 10, 2021 at 5:18 AM atarax ***@***.***> wrote:
Contributors.rst
i don't see a file with that name? oO
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#581 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACE2MMYEW23UYSCEBDHGXEDS6JMP7ANCNFSM4XGNC6FQ>
.
|
2fa5f0c
to
a2cfbc7
Compare
no worries, i know that kind of issues :) |
Thanks for doing this with clean git history too! Awesome work @atarax ! |
You're welcome! :) |
Summary of Changes
This PR will add three fundamental-data endpoints
@hydrosquall
Checklist
HISTORY.rst
with an entry summarizing your change