Skip to content

Stock_Information Issue #789 - #805

Open
KazushiR wants to merge 1 commit into
python-geeks:mainfrom
KazushiR:Stock_Information
Open

KazushiR wants to merge 1 commit into
python-geeks:mainfrom
KazushiR:Stock_Information

Conversation

@KazushiR

@KazushiR KazushiR commented Nov 4, 2021

Copy link
Copy Markdown
Contributor

Pull Request Template

script name -
Stock_Infromation.py

Brief about script
Pulls the top stock from a subreddit of Wallstreetbets. From there, it categorizes the top 5 stocks then, it pulls some information about the stock in an SQL database.

Issue no.(must) - #789

Self Check(Tick After Making pull Request)

  • [ x] This issue was assigned to me.
  • One Change in one Pull Request
  • [ x] My file is in proper folder (Name of folder should be in lowercase with no space in between) (E.g. meet_schedular)
  • [ x] I am following clean code and Documentation and my code is well linted with flake8.
  • [x ] I have added README.md and requirements.txt (Include version numbers too e.g. pandas==0.0.1) with my script
  • [x ] I have used REPO README TEAMPLATE (Necessary)
  • [ x] Just including required dependencies in requirements.txt (Don't include Python version too)

If issue was not assigned to you Please don't make a PR. It will marked as invalid.

@KazushiR

KazushiR commented Nov 4, 2021 •

Copy link
Copy Markdown
Contributor Author

I put the code through FLAKE8 and it looked fine. Please let me know if any changes needs to be put in place. @pawangeek

@pawangeek pawangeek linked an issue Nov 9, 2021 that may be closed by this pull request
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.

stock info tracker

1 participant