Skip to content
This repository was archived by the owner on Nov 18, 2024. It is now read-only.

added discord status option #140

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Frain-Breeze
Copy link

added the option to disable the "downloaded xxxx images" as discord status.
I personally don't like the status thingy very much, so I thought I might as well add it as an option.

added the option to disable the "downloaded xxxx images" as discord status.
@@ -98,6 +98,7 @@ func main() {
cfg.Section("general").NewKey("max download retries", "5")
cfg.Section("general").NewKey("download timeout", "60")
cfg.Section("general").NewKey("send notices to interactive channels", "false")
cfg.Section("general").NewKey("set downloaded image count as discord status", "true")

Choose a reason for hiding this comment

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

Error return value of (*github.com/Seklfreak/discord-image-downloader-go/vendor/gopkg.in/ini.v1.Section).NewKey is not checked (from errcheck)

Copy link
Author

Choose a reason for hiding this comment

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

I don't see the problem here. Is it a false positive?

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Found some fixes!

P.S. share your ideas, feedbacks or issues with us at https://github.com/fixmie/feedback (this message will be removed after the beta stage).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants