Skip to content

Feat: Inspect dart-runner executable - #139

Merged
HafeezOJ merged 8 commits into
mainfrom
134-add-warning-at-run-start-if-dart-runner-is-missing
Jul 21, 2026
Merged

Feat: Inspect dart-runner executable#139
HafeezOJ merged 8 commits into
mainfrom
134-add-warning-at-run-start-if-dart-runner-is-missing

Conversation

@HafeezOJ

Copy link
Copy Markdown
Collaborator

Description

This PR checks the availability of the dart-runner executable. It adds a warning to the logs if the dart-runner executable is not available or if the version of the dart-runner executable is lower than 1. An error is added to the log when an item is not bagged due to the unavailability of dart-runner.

See #134

Documentation Update

  • I have updated README.md and other relevant documentation
  • No documentation update is needed

Implementation Notes

  • A function to inspect dart-runner executable has been added to Utils.py
  • Check if dart-runner executable is available during preliminary checks
  • Prevents an attempt to bag any item when dart-runner executable is not available.
  • Final counts are not modified in this PR since processed items will be 0 if dart-runner executable is not available. Only warnings and errors are logged.

@HafeezOJ HafeezOJ linked an issue Jul 21, 2026 that may be closed by this pull request
1 task
@HafeezOJ
HafeezOJ marked this pull request as ready for review July 21, 2026 00:32
@HafeezOJ
HafeezOJ requested a review from zoidy July 21, 2026 00:32
Comment thread figshare/Article.py
@HafeezOJ
HafeezOJ force-pushed the 134-add-warning-at-run-start-if-dart-runner-is-missing branch from 705f7c5 to 8b0c13e Compare July 21, 2026 16:01
@HafeezOJ HafeezOJ self-assigned this Jul 21, 2026
@HafeezOJ
HafeezOJ requested a review from zoidy July 21, 2026 16:04
@HafeezOJ
HafeezOJ merged commit c544b43 into main Jul 21, 2026
1 check passed
@HafeezOJ HafeezOJ added the enhancement New feature or request label Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add warning at run start if dart-runner is missing

2 participants