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

Analyst Ratings #106

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

renfrowjay
Copy link

Description

Changed names of some functions and variables to align with the wording used on the Finviz website:
outer ratings to analyst ratings
inside trader to insider trading
fundament to fundamentals

Type of change

  • Code styling

What you did

In quote.py renamed some methods:
renamed ticker_fundament to ticker_fundamentals
renamed ticker_outer_ratings to ticker_analyst_ratings
renamed ticker_inside_trader to ticker_insider_trading
In quote.py renamed some variables:
renamed fundament to fundamentals
renamed fundament_info to fundamentals_info
renamed fundament_table to fundamentals_table
renamed fullview_ratings_outer to fullview_ratings_analyst
renamed outer to analyst
renamed inside_trader to insider_trading
In quote.py renamed some dictionary keys:
in info renamed fundament to fundamentals
in info renamed ratings_outer to ratings_analyst
in info renamed inside_trader to insider_trading
in info_dict renamed Outer to Analyst
Tried to find all occurrences of these changes in other files and updated.

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.

None yet

1 participant