-
Notifications
You must be signed in to change notification settings - Fork 2
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.
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
Implement channel_id command #2
Open
aninhasalesp
wants to merge
22
commits into
PythonicCafe:feature/cli
Choose a base branch
from
aninhasalesp:feature/channel-id
base: feature/cli
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Implement channel_id command #2
aninhasalesp
wants to merge
22
commits into
PythonicCafe:feature/cli
from
aninhasalesp:feature/channel-id
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Implemented method to convert a list of dictionaries into a CSV file or string;
…RLs or a CSV file containing URLs; - Added commands directory structure
fix: show error with parser if not in debug mode
- Changed file path passing to use from in the method;
…and return a string; - Added logic to convert values retrieved from the CSV file to strings before appending them to the data list;
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add command to extract youtube channel IDs from URLs and generate CSV
Overview
This PR introduces the ChannelId command, which extracts YouTube channel IDs from URLs or from a CSV file containing URLs and generates a CSV file with the channel IDs. Additionally, it refactors the CLI tool to use a command-based structure, with a base Command class to standardize command implementation.
Features
ChannelId
command to handle the extraction of youtube channel IDs.execute
method to handle the end-to-end process of fetching channel IDs and generating the output CSV.Command
as a foundational class for defining CLI commands.parse_arguments
method to configure argument parsing and setexecute
method as the default action for each command.execute
method, to be overridden by subclasses, for executing command-specific logic.Usage
To use the new
ChannelId
command:From a list of URLs:
These are other ways to run the ChannelId command: