Skip to content

Quick fixes and improved doc#4

Open
haridsv wants to merge 2 commits into
mlschmitt:mainfrom
haridsv:quick-fixes
Open

Quick fixes and improved doc#4
haridsv wants to merge 2 commits into
mlschmitt:mainfrom
haridsv:quick-fixes

Conversation

@haridsv
Copy link
Copy Markdown

@haridsv haridsv commented Jan 28, 2024

  • Skip potentially uninteresting tweets by catching and ignoring some exceptions
  • Create parent directories while generating html, use pathlib.Path for the ease.
  • Rename config.json to config_template.json and update doc to create it. Don't track config.json so that it avoids the possibility of some developer accidentally publishing their local changes with twitter auth info.
  • Updated doc with more details on how to find the required info.

@ewoks
Copy link
Copy Markdown

ewoks commented Sep 15, 2024

@mlschmitt can we take care of this first please as @haridsv already made some effort?
I will take my PR #9 back in the meantime

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.

2 participants