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.

Sign up for GitHub

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

Jump to bottom

Print video information in the CLI #951

Merged
merged 28 commits into from
Aug 1, 2024
Merged

Print video information in the CLI #951

merged 28 commits into from
Aug 1, 2024

Conversation

ScrubN
Copy link
Collaborator

@ScrubN ScrubN commented Jan 20, 2024 •
edited
Loading

Still a WIP. Feedback is welcomed.

TODO:

  • Establish a JSON standard for scripts to parse the output
  • Also print Clip/Video metadata? (i.e. name, length, category, streamer, etc.)
    • Should this be implemented by a different verb?
      • If no, should the verb be changed from streaminfo to something else?
  • Try to find a versatile solution to the inaccurate framerates that Twitch's API started returning
  • Is it worth having a custom Table.cs instead of just using Spectre.Console?

@ScrubN ScrubN changed the title Get video information from the CLI Print video information in the CLI Jan 20, 2024
@ScrubN ScrubN force-pushed the cli-stream-info branch 2 times, most recently from 1399490 to d099ebb Compare July 17, 2024 02:18
@ScrubN ScrubN force-pushed the cli-stream-info branch from d099ebb to 32b32df Compare July 17, 2024 02:48
ScrubN added 21 commits July 16, 2024 23:03
@ScrubN
Make M3U8.Metadata properties init
4b12533
@ScrubN
Refactor
25a1de4
@ScrubN
Use progress reporter where appropriate
e813896
@ScrubN
Provide at least 3 digits when stringifying byte count
ffd9ca9
@ScrubN
Fetch clip curator & clip/vod broadcaster login
0ceb748
@ScrubN
Switch from TwitchDownloaderCLI.Tools.Table to Spectre.Console.Table
54841eb
@ScrubN
Hide JSON format from help text
972b493
@ScrubN
Extract code into dedicated methods
40a6315
@ScrubN
Cleanup
11a494d
@ScrubN
More cleanup
fe4f587
@ScrubN
Merge branch 'master' of https://github.com/lay295/TwitchDownloader i…
43e9dc5 
…nto cli-stream-info
@ScrubN
Fix tests
5d0232e
@ScrubN
Add video chapter table
e74c32d
@ScrubN
Oops
5025b63
@ScrubN
Better timestamp strings
9b36f98
@ScrubN
Display ASCII login for users with non-ASCII usernames, cleanup
f80e2c9
@ScrubN
Ensure output encoding is UTF-8
53b87a4
@ScrubN
Add README entry
6fe74c0
@ScrubN
StreamInfo -> Info
60790df
@ScrubN
Only link user page if login is present
717e3ae
@ScrubN
Fix NRE
ded72f5
@ScrubN ScrubN marked this pull request as ready for review August 1, 2024 19:40
@ScrubN ScrubN merged commit d0c48db into lay295:master Aug 1, 2024
@ScrubN ScrubN deleted the cli-stream-info branch August 1, 2024 19:52
@ScrubN
Copy link
Collaborator Author

ScrubN commented Aug 1, 2024

Should have named it Support printing video information in the CLI, whoops :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CLI Tool - Any way to get list of qualities?
1 participant
@ScrubN