• [Study] Python telegram bot + Flask

  身為 要出錢去平台開機器的苦主 我希望 notifier_web.py 和 pt_bot.py 都能使用 Flask 來接收 http request 如此 我就不用開兩台機器分別執行他們

  Definition of Done

  application 可以同時接受 LINE notify & Telegram Bot Webhook

  opened by chen-tf 5
• [Study] PChome 商品資訊 - 爬蟲

  身為 使用者 我希望 有更多的電商平台商品可以收藏 如此 我可以有更多的機會買到便宜商品

  背景

  現行只有支援 momo shop，需要知道 PChome 是否也有可以單靠 http request，不用 javascript render 就可以得到商品狀態的方法

  Definition of Done

  可以透過 PChome 商品頁 URL 得到以下資訊

  1. 商品名稱
  2. 價格
  3. 上/下架狀態

  opened by chen-tf 5
• [Study] GitHub Action 和 Fly.io 整合

  身為 開發者 我希望 main branch changed 的時候，會自動部署到對應環境 如此 我才不用手動部署

  背景

  目前都是自己在 local 透過 fly.io CLI tool 的方式去部署

  Definition of Done

  1. 順利在 code merge main branch 的時候將服務部署上去

  opened by chen-tf 4
• [Study] 採用 ORM framework

  身為 開發者 我希望 資料持久化的操作可以更簡單 如此 才可以專心在邏輯的操作上

  背景

  目前的 pt_repository 使用的是 psycopg2 和他的 conneciton pool 非常的痛苦，常常會有需要自行檢查是否斷線的問題，想要直接換上一套 ORM framework 在往後的資料操作上才可以更輕鬆。

  Definition of Done

  1. 要使用什麼 python ORM framework
  2. 如何把目前的 code migrate 過去
  3. example code

  opened by chen-tf 4
• Feature/build linter

  Why need this change? / Root cause:

  • #17
  • 加速專案的維護性及協作性
  • 統一程式碼排版及風格

  Changes made:

  • update README.md file
  • update requirements.txt file
  • add .pylintrc file
  • add .pre-commit-config.yaml file
  • add linters.yml file under .github/workflows directory

  Test Scope / Change impact:

  • CI/CD (linters.yml) should work and check every pull request

  issue #17

  opened by w305jack 2
• [Study] 採用 ORM framework

  Why need this change? / Root cause:

  • #7
  • 把現有的table轉成ORM的model格式

  Changes made:

  • models.py：對應到create_tables.sql的3張表(partial index沒建，其他都先參照原本的屬性)
  • tutorial.md：存放相關技術文件
  • /migrations：flask-migrate後的產物

  Test Scope / Change impact:

  1. 改用測試的DB

  # repository/app.py
  app.config['SQLALCHEMY_DATABASE_URI'] = \
      f"postgresql://{pt_config.DB_USER}:{pt_config.DB_PASSWORD}@{pt_config.DB_HOST}:5432/test"
  

  2. 在postgres中新增test database

  >> psql -U postgres
  >> create database test;
  

  3. 創建tables

  flask --app repository/models.py db upgrade
  

  opened by s9891326 2
• Bump pytest-cov from 3.0.0 to 4.0.0

  Bumps pytest-cov from 3.0.0 to 4.0.0.

  Changelog

  Sourced from pytest-cov's changelog.

  4.0.0 (2022-09-28)

  Note that this release drops support for multiprocessing.

  • --cov-fail-under no longer causes pytest --collect-only to fail Contributed by Zac Hatfield-Dodds in [#511](https://github.com/pytest-dev/pytest-cov/issues/511) <https://github.com/pytest-dev/pytest-cov/pull/511>_.

  • Dropped support for multiprocessing (mostly because issue 82408 <https://github.com/python/cpython/issues/82408>_). This feature was mostly working but very broken in certain scenarios and made the test suite very flaky and slow.

    There is builtin multiprocessing support in coverage and you can migrate to that. All you need is this in your .coveragerc::

    [run] concurrency = multiprocessing parallel = true sigterm = true

  • Fixed deprecation in setup.py by trying to import setuptools before distutils. Contributed by Ben Greiner in [#545](https://github.com/pytest-dev/pytest-cov/issues/545) <https://github.com/pytest-dev/pytest-cov/pull/545>_.

  • Removed undesirable new lines that were displayed while reporting was disabled. Contributed by Delgan in [#540](https://github.com/pytest-dev/pytest-cov/issues/540) <https://github.com/pytest-dev/pytest-cov/pull/540>_.

  • Documentation fixes. Contributed by Andre Brisco in [#543](https://github.com/pytest-dev/pytest-cov/issues/543) <https://github.com/pytest-dev/pytest-cov/pull/543>_ and Colin O'Dell in [#525](https://github.com/pytest-dev/pytest-cov/issues/525) <https://github.com/pytest-dev/pytest-cov/pull/525>_.

  • Added support for LCOV output format via --cov-report=lcov. Only works with coverage 6.3+. Contributed by Christian Fetzer in [#536](https://github.com/pytest-dev/pytest-cov/issues/536) <https://github.com/pytest-dev/pytest-cov/issues/536>_.

  • Modernized pytest hook implementation. Contributed by Bruno Oliveira in [#549](https://github.com/pytest-dev/pytest-cov/issues/549) <https://github.com/pytest-dev/pytest-cov/pull/549>_ and Ronny Pfannschmidt in [#550](https://github.com/pytest-dev/pytest-cov/issues/550) <https://github.com/pytest-dev/pytest-cov/pull/550>_.

  Commits

  • 28db055 Bump version: 3.0.0 → 4.0.0
  • 57e9354 Really update the changelog.
  • 56b810b Update chagelog.
  • f7fced5 Add support for LCOV output
  • 1211d31 Fix flake8 error
  • b077753 Use modern approach to specify hook options
  • 00713b3 removed incorrect docs on data_file.
  • b3dda36 Improve workflow with a collecting status check. (#548)
  • 218419f Prevent undesirable new lines to be displayed when report is disabled
  • 60b73ec migrate build command from distutils to setuptools
  • Additional commits viewable in compare view

  
  

  

  Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

  ---

  Dependabot commands and options

  
  

  You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

  dependencies 

  opened by dependabot[bot] 2
• [Study] 調整 readme

  身為 對這個專案有興趣的開發者 我希望 專案有清楚的 readme 如此 才可以快速地了解這個專案的背景

  背景

  目前的 readme 比較像是一個故事書...，需要思考一下該做一些什麼調整，讓未來想要 fork project 的人能快速地進入狀態。

  Definition of Done

  1. 如何執行這個專案
  2. 想要擴充功能要從何進行
  3. 實際上支援幾種指令

  opened by chen-tf 2
• [Dev] 導入 SQLAlchemy 並改寫 pt_repository 的 upsert_user

  Why need this change? / Root cause:

  • #55

  Changes made:

  • repository/models.py：對應到create_tables.sql的3張表(index沒建，其他都先參照原本的屬性)
  • repository/database.py：創建sqlalchemy跟postgresql的連線
  • repository/init.py：各個model的CRUD行為
  • pt_service.py：更改原本的upsert_user()
  • .pylintrc：增加import-outside-toplevel，因為pt_service裡面有一些func底下有import pt_bot的行為，如果抽到最上面import，那跑app.py的時候就會重複啟用bot了，或者要把那邊的程式解偶一下

  Test Scope / Change impact:

  # 原本
  def upsert_user(user_id, chat_id):
      return pt_repository.upsert_user(user_id, chat_id)
  
  # 改後
  def upsert_user(user_id, chat_id):
      return repository.upsert_user(user_id, chat_id)
  

  opened by s9891326 1
• Bump coverage from 6.3.3 to 7.0.2

  Bumps coverage from 6.3.3 to 7.0.2.

  Changelog

  Sourced from coverage's changelog.

  Version 7.0.2 — 2023-01-02

  • Fix: when using the [run] relative_files = True setting, a relative [paths] pattern was still being made absolute. This is now fixed, closing issue 1519_.

  • Fix: if Python doesn't provide tomllib, then TOML configuration files can only be read if coverage.py is installed with the [toml] extra. Coverage.py will raise an error if TOML support is not installed when it sees your settings are in a .toml file. But it didn't understand that [tools.coverage] was a valid section header, so the error wasn't reported if you used that header, and settings were silently ignored. This is now fixed, closing issue 1516_.

  • Fix: adjusted how decorators are traced on PyPy 7.3.10, fixing issue 1515_.

  • Fix: the coverage lcov report did not properly implement the --fail-under=MIN option. This has been fixed.

  • Refactor: added many type annotations, including a number of refactorings. This should not affect outward behavior, but they were a bit invasive in some places, so keep your eyes peeled for oddities.

  • Refactor: removed the vestigial and long untested support for Jython and IronPython.

  .. _issue 1515: nedbat/coveragepy#1515 .. _issue 1516: nedbat/coveragepy#1516 .. _issue 1519: nedbat/coveragepy#1519

  .. _changes_7-0-1:

  Version 7.0.1 — 2022-12-23

  • When checking if a file mapping resolved to a file that exists, we weren't considering files in .whl files. This is now fixed, closing issue 1511_.

  • File pattern rules were too strict, forbidding plus signs and curly braces in directory and file names. This is now fixed, closing issue 1513_.

  • Unusual Unicode or control characters in source files could prevent reporting. This is now fixed, closing issue 1512_.

  • The PyPy wheel now installs on PyPy 3.7, 3.8, and 3.9, closing issue 1510_.

  .. _issue 1510: nedbat/coveragepy#1510 .. _issue 1511: nedbat/coveragepy#1511

  ... (truncated)

  Commits

  • 2f731e2 docs: sample HTML
  • dbbd5b7 docs: prep for 7.0.2
  • d08e6d0 fix: relative_files should keep relative path maps. #1519
  • 3f0bce2 mypy: partial debug.py and pytracer.py
  • ffc701a mypy: test_xml.py
  • 5580cf8 mypy: xmlreport.py
  • 0c9b5e0 mypy: check collector.py and plugin_support.py
  • 8f4d404 refactor: a better way to filter coverage debug pybehave
  • a3f3841 mypy: add cmdline.py and test_cmdline.py
  • 09f9188 mypy: add env.py
  • Additional commits viewable in compare view

  
  

  

  Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

  ---

  Dependabot commands and options

  
  

  You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

  dependencies 

  opened by dependabot[bot] 1
• Bump coverage from 6.3.3 to 7.0.1

  Bumps coverage from 6.3.3 to 7.0.1.

  Changelog

  Sourced from coverage's changelog.

  Version 7.0.1 — 2022-12-23

  • When checking if a file mapping resolved to a file that exists, we weren't considering files in .whl files. This is now fixed, closing issue 1511_.

  • File pattern rules were too strict, forbidding plus signs and curly braces in directory and file names. This is now fixed, closing issue 1513_.

  • Unusual Unicode or control characters in source files could prevent reporting. This is now fixed, closing issue 1512_.

  • The PyPy wheel now installs on PyPy 3.7, 3.8, and 3.9, closing issue 1510_.

  .. _issue 1510: nedbat/coveragepy#1510 .. _issue 1511: nedbat/coveragepy#1511 .. _issue 1512: nedbat/coveragepy#1512 .. _issue 1513: nedbat/coveragepy#1513

  .. _changes_7-0-0:

  Version 7.0.0 — 2022-12-18

  Nothing new beyond 7.0.0b1.

  .. _changes_7-0-0b1:

  Version 7.0.0b1 — 2022-12-03

  A number of changes have been made to file path handling, including pattern matching and path remapping with the [paths] setting (see :ref:config_paths). These changes might affect you, and require you to update your settings.

  (This release includes the changes from 6.6.0b1 <changes_6-6-0b1_>_, since 6.6.0 was never released.)

  • Changes to file pattern matching, which might require updating your configuration:

    • Previously, * would incorrectly match directory separators, making precise matching difficult. This is now fixed, closing issue 1407_.

    • Now ** matches any number of nested directories, including none.

  • Improvements to combining data files when using the

  ... (truncated)

  Commits

  • c5cda3a docs: releases take a little bit longer now
  • 9d4226e docs: latest sample HTML report
  • 8c77758 docs: prep for 7.0.1
  • da1b282 fix: also look into .whl files for source
  • d327a70 fix: more information when mapping rules aren't working right.
  • 35e249f fix: certain strange characters caused reporting to fail. #1512
  • 152cdc7 fix: don't forbid plus signs in file names. #1513
  • 31513b4 chore: make upgrade
  • 873b059 test: don't run tests on Windows PyPy-3.9
  • 5c5caa2 build: PyPy wheel now installs on 3.7, 3.8, and 3.9. #1510
  • Additional commits viewable in compare view

  
  

  

  Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

  ---

  Dependabot commands and options

  
  

  You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

  dependencies 

  opened by dependabot[bot] 1
• Bump coverage from 6.3.3 to 7.0.3

  Bumps coverage from 6.3.3 to 7.0.3.

  Changelog

  Sourced from coverage's changelog.

  Version 7.0.3 — 2023-01-03

  • Fix: when using pytest-cov or pytest-xdist, or perhaps both, the combining step could fail with assert row is not None using 7.0.2. This was due to a race condition that has always been possible and is still possible. In 7.0.1 and before, the error was silently swallowed by the combining code. Now it will produce a message "Couldn't combine data file" and ignore the data file as it used to do before 7.0.2. Closes issue 1522_.

  .. _issue 1522: nedbat/coveragepy#1522

  .. _changes_7-0-2:

  Version 7.0.2 — 2023-01-02

  • Fix: when using the [run] relative_files = True setting, a relative [paths] pattern was still being made absolute. This is now fixed, closing issue 1519_.

  • Fix: if Python doesn't provide tomllib, then TOML configuration files can only be read if coverage.py is installed with the [toml] extra. Coverage.py will raise an error if TOML support is not installed when it sees your settings are in a .toml file. But it didn't understand that [tools.coverage] was a valid section header, so the error wasn't reported if you used that header, and settings were silently ignored. This is now fixed, closing issue 1516_.

  • Fix: adjusted how decorators are traced on PyPy 7.3.10, fixing issue 1515_.

  • Fix: the coverage lcov report did not properly implement the --fail-under=MIN option. This has been fixed.

  • Refactor: added many type annotations, including a number of refactorings. This should not affect outward behavior, but they were a bit invasive in some places, so keep your eyes peeled for oddities.

  • Refactor: removed the vestigial and long untested support for Jython and IronPython.

  .. _issue 1515: nedbat/coveragepy#1515 .. _issue 1516: nedbat/coveragepy#1516 .. _issue 1519: nedbat/coveragepy#1519

  .. _changes_7-0-1:

  Version 7.0.1 — 2022-12-23

  ... (truncated)

  Commits

  • 2ff9098 docs: prep for 7.0.3
  • 1f34d8b fix: race condition on data file shouldn't break combining. #1522
  • 85170bf build: two-step combines for speed
  • 1605f07 mypy: misc.py, test_misc.py
  • 4f3ccf2 refactor: a better way to have maybe-importable third-party modules
  • 98301ed mypy: test_config.py, test_context.py
  • 9d2e1b0 mypy: test_concurrency.py, test_python.py
  • c3ee30c refactor(test): use tmp_path instead of tmpdir
  • 0b05b45 mypy: test_annotate.py test_arcs.py test_collector.py
  • 2090f79 style: better
  • Additional commits viewable in compare view

  
  

  

  Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

  ---

  Dependabot commands and options

  
  

  You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

  dependencies 

  opened by dependabot[bot] 0
• Bump pytest-html from 3.1.1 to 3.2.0

  Bumps pytest-html from 3.1.1 to 3.2.0.

  Release notes

  Sourced from pytest-html's releases.

  v3.2.0

  Changes

  • Revert "Rename master branch to main" (#562) @​BeyondEvil
  • Release v3.2.0 (#556) @​BeyondEvil
  • Add py as dependency (#555) @​smartEBL
  • pip isn't a build dependency (#516) @​rossburton
  • Migrate to precommit.ci (#487) @​BeyondEvil
  • Use the tests reusable workflow (#486) @​BeyondEvil

  Minor Changes

  • Allow for redacting of environment table values (#431) @​gnikonorov
  • Implement the visible URL query parameter to control visibility of test results on page load. (#433) @​gnikonorov
  • Make the report tab title reflect the report name (#420) @​gnikonorov

  Bugfixes

  • Rename master branch to main (#560) @​ssbarnea
  • chore: Deactivate broken test for now (#557) @​BeyondEvil
  • Use pathlib.Path for file path handling (#514) @​mrlegohead0x45
  • Re-enable tests against latest pytest (#421) @​BeyondEvil

  Changelog

  Sourced from pytest-html's changelog.

  3.2.0 (2022-10-25)

  
  * Explicitly add py.xml dependency.
  
  
  Thanks to @smartEBL <https://github.com/smartEBL>_ for the PR
  
  
  Implement the visible URL query parameter to control visibility of test results on page load. ([#399](https://github.com/pytest-dev/pytest-html/issues/399) <https://github.com/pytest-dev/pytest-html/issues/399>_)
  
  Thanks to @TheCorp <https://github.com/TheCorp>_ for reporting and @gnikonorov <https://github.com/gnikonorov>_ for the fix
  
  
  
  Make the report tab title reflect the report name. ([#412](https://github.com/pytest-dev/pytest-html/issues/412) <https://github.com/pytest-dev/pytest-html/issues/412>_)
  
  Thanks to @gnikonorov <https://github.com/gnikonorov>_ for the PR
  
  
  
  Implement :code:environment_table_redact_list to allow for redaction of environment table values. ([#233](https://github.com/pytest-dev/pytest-html/issues/233) <https://github.com/pytest-dev/pytest-html/issues/233>_)
  
  Thanks to @fenchu <https://github.com/fenchu>_ for reporting and @gnikonorov <https://github.com/gnikonorov>_ for the PR
  

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.