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

【改善要望】batch用スクリプトの共通クラス化・設定ファイルのjson化 #15

Open
2 tasks
shinGangan opened this issue Jan 22, 2023 · 0 comments · May be fixed by #17
Open
2 tasks
Assignees

Comments

@shinGangan
Copy link
Collaborator

内容

  • 現在 /batch/firebaase直下にpythonスクリプトが配置されている。各グループごとにファイル・関数が用意されているが、共通化・クラス化出来る箇所が多い。そこで、クラス化したい
  • 画像取得用pathを.txt形式で保持しているファイルが散見される。そこで、.json‘化または.ini‘形式に置き換えたい

改善案

  • /batch直下のpythonスクリプトのクラス化・共通化
    • /firebase‘、/scraping`直下は本イシューでは対応しない。対応する場合は別イシューで対応する。
  • 設定ファイルの.json化または.ini
    • あまり可変しない設定情報は設定ファイルとして保持した方が良い。

本イシューで対応しないこと

  • /firebase‘、/scraping`直下のリファクタリング
  • pytest導入: /batch直下への導入は検討中である。

TODO

  • {group_name}_blog_latest.pyのリファクタリング
  • /{group_name}/blogUrls.txtの設定ファイル化
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant