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.

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

Change from -m <mode> syntax to <verb>, update gitignore #4

Merged
merged 2 commits into from
Dec 12, 2022

Conversation

ScrubN
Copy link
Contributor

@ScrubN ScrubN commented Nov 19, 2022

Very soon TwitchDownloaderCLI will switch from ./TwitchDownloaderCLI -m <mode> to ./TwitchDownloaderCLI <verb>. While the -m syntax will still work, it prints a warning message which you likely won't want showing to users. So only when the latest release is greater than 1.50.6 merge this.

Also added .vs folder to gitignore


Wish I could help more with this project, like adding per-service emote embedding, but I don't know enough C

@mohad12211
Copy link
Owner

Thank you!

Wish I could help more with this project, like adding per-service emote embedding, but I don't know enough C

I believe that requires some GUI changes, I will implement that soon.

@ScrubN
Copy link
Contributor Author

ScrubN commented Nov 19, 2022

I believe that requires some GUI changes, I will implement that soon.

It should just be adding 3 checkboxes to the GUI that append the correct arguments to the command string. I put an example on disabling per-service 3rd party emotes in the CLI documentation

@ScrubN
Copy link
Contributor Author

ScrubN commented Nov 20, 2022

--embed-emotes will likely become --embed-images. -E will remain the same though

@ScrubN
Copy link
Contributor Author

ScrubN commented Nov 20, 2022

This is now safe to be merged!

@mohad12211 mohad12211 merged commit 8ae9054 into mohad12211:master Dec 12, 2022
@ScrubN ScrubN deleted the patch-1 branch December 29, 2022 04:26
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.

2 participants