Skip to content

chore: remove unused api_pull entrypoint - #57

Merged
cchwala merged 1 commit into
mainfrom
chore/remove-api-pull-entrypoint
Jul 13, 2026
Merged

chore: remove unused api_pull entrypoint#57
cchwala merged 1 commit into
mainfrom
chore/remove-api-pull-entrypoint

Conversation

@cchwala

@cchwala cchwala commented Jul 13, 2026

Copy link
Copy Markdown
Member

The api_fetcher container handles HTTP/API polling by writing JSON files to the shared incoming volume, which the parser then processes. This file-based approach provides consistent archiving for cold storage.

Removals:

  • parser/entrypoints/api_pull.py (stub entrypoint)
  • 'api_pull' from valid_entrypoints in generate_config.py
  • 'api_pull' from entrypoint documentation in users.yml

Retained:

  • sftp_pull entrypoint stub (still planned for remote SFTP polling)

Tests: 90 passing

The api_fetcher container handles HTTP/API polling by writing JSON files
to the shared incoming volume, which the parser then processes. This file-based
approach provides consistent archiving for cold storage.

Removals:
- parser/entrypoints/api_pull.py (stub entrypoint)
- 'api_pull' from valid_entrypoints in generate_config.py
- 'api_pull' from entrypoint documentation in users.yml

Retained:
- sftp_pull entrypoint stub (still planned for remote SFTP polling)

Tests: 90 passing
@codecov

codecov Bot commented Jul 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.10%. Comparing base (0fc99da) to head (e3df341).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #57      +/-   ##
==========================================
+ Coverage   86.07%   86.10%   +0.02%     
==========================================
  Files          40       39       -1     
  Lines        3419     3418       -1     
==========================================
  Hits         2943     2943              
+ Misses        476      475       -1     
Flag Coverage Δ
mno_simulator 86.12% <ø> (ø)
parser 92.00% <ø> (+0.04%) ⬆️
scripts 74.21% <100.00%> (ø)
webserver 73.78% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cchwala
cchwala merged commit 6d03b11 into main Jul 13, 2026
8 checks passed
@cchwala
cchwala deleted the chore/remove-api-pull-entrypoint branch July 13, 2026 19:30
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