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

[WIP] Refactor code to gem #60

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Conversation

athityakumar
Copy link
Owner

Contribution

Refactors the entire codebase with proper community standards.

  • Relevant Issues : (none)
  • Relevant PRs : (none)
  • Type of change :
    • New TV series
    • Bug fix for existing feature
    • Code quality improvement
    • New feature
    • Addition or Improvement of documentation

@athityakumar athityakumar force-pushed the refactor-code-to-gem branch 2 times, most recently from 73de58e to 924043a Compare March 3, 2018 04:41
IMDB_LINK = 'http://www.imdb.com/title/tt4532368/'.freeze
SCRAPE_LINK = 'https://en.wikipedia.org/wiki/List_of_Legends_of_Tomorrow_episodes'.freeze
JSON_FILE_PATH = File.join(TVSeries::SERIES_JSON_PATH, 'legends_of_tomorrow.json').freeze
SHORT_NAME = 'lot'.freeze
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For all scraper classes, store just the series_name (short name) and access other details from the base.json.

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.

1 participant