Skip to content

Support 100 hours long videos - #2

Merged
CGuichard merged 1 commit into
mainfrom
fix/ffmpeg-parsing
Jan 31, 2026
Merged

Support 100 hours long videos#2
CGuichard merged 1 commit into
mainfrom
fix/ffmpeg-parsing

Conversation

@CGuichard

Copy link
Copy Markdown
Owner

Close #1

Support more than two numbers for hours and milliseconds part in duration and status text.

@CGuichard CGuichard self-assigned this Jan 30, 2026
@CGuichard CGuichard added the bug Something isn't working label Jan 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Audit dependencies

pip-audit

No known vulnerabilities found

libyear

Your system is up-to-date!

@github-actions

Copy link
Copy Markdown
Contributor

Test Results

  5 files  ±0    5 suites  ±0   2s ⏱️ ±0s
 30 tests ±0   30 ✅ ±0  0 💤 ±0  0 ❌ ±0 
150 runs  ±0  150 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 892f86a. ± Comparison against base commit 5bfa5df.

@github-actions

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Details
Filename                   Stmts    Miss  Cover    Missing
-----------------------  -------  ------  -------  ------------
src/pffmpeg/__init__.py        5       0  100.00%
src/pffmpeg/_args.py          23       0  100.00%
src/pffmpeg/_cli.py            6       0  100.00%
src/pffmpeg/_runner.py        91      21  76.92%   48-64, 72-76
src/pffmpeg/_utils.py         21       0  100.00%
TOTAL                        146      21  85.62%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: 892f86a

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@CGuichard
CGuichard merged commit a8866cf into main Jan 31, 2026
13 checks passed
@CGuichard
CGuichard deleted the fix/ffmpeg-parsing branch January 31, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Duration not parsed when at least 100 hours long

1 participant