search

cedws / discord-delete

Tool to delete Discord message history
GNU General Public License v3.0
126 stars 18 forks source link

Add option to skip pinned messages #43

Closed mrrfv closed 2 years ago

mrrfv commented 2 years ago

Fixes #42. This is my first time using Go, so lmk if there's something that needs to be changed/fixed before merging.

cedws commented 2 years ago

Hey, looks good, thank you. Go uses tabs universally so could you change your editor settings? Or you can run gofmt -s -w . to autofix.

I am on holiday at the moment so won't be able to merge for a few days.

mrrfv commented 2 years ago

gofmt -s -w . doesn't seem to do anything. I assume the command intentionally doesn't output anything, but neither Visual Studio Code nor Git doesn't detect any changes.

Could you clone my fork, switch to the patch-1 branch, and try running the command yourself once your holidays end? Edits by maintainers are enabled, so you should have write access.

cedws commented 2 years ago

Actually it's all fine, GitHub mobile is showing the diff a bit strangely. I guess VSCode ran gofmt for you.

I'll get back you soon.

cedws commented 2 years ago

Nice feature, thanks again.