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

Support no tracker tags and extra tags on race existing #45

Merged
merged 3 commits into from
Jan 9, 2024

Conversation

fireph
Copy link
Contributor

@fireph fireph commented Jan 7, 2024

The flags --no-tracker-tags and --extra-tags were only supported with qbit-race add and not with qbit-race race. This PR adds this feature to qbit-race race with the same syntax. I didn't add support for setting the category because it doesn't really make sense to change the category after the torrent has already started.

@ckcr4lyf
Copy link
Owner

ckcr4lyf commented Jan 9, 2024

Thanks for the MR. Let me test it offline once.

Could you remove the changes made in package-lock ? I will update the versioning myself when I make the release.

Copy link
Owner

@ckcr4lyf ckcr4lyf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed externally via #46 . Thanks!

package-lock.json Outdated Show resolved Hide resolved
package-lock.json Outdated Show resolved Hide resolved
@fireph
Copy link
Contributor Author

fireph commented Jan 9, 2024

Thanks for the MR. Let me test it offline once.

Could you remove the changes made in package-lock ? I will update the versioning myself when I make the release.

updated to remove the package-lock changes!

@ckcr4lyf
Copy link
Owner

ckcr4lyf commented Jan 9, 2024

LGTM, thanks.

@ckcr4lyf ckcr4lyf merged commit f85b498 into ckcr4lyf:master Jan 9, 2024
2 checks passed
@ckcr4lyf ckcr4lyf mentioned this pull request Jan 9, 2024
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