make 'magit-todos-exclude-globs' a safe local variable

[wsw0108]

otherwise, it will always ask whether to apply this local variable

[wsw0108]

I can make this variable as safe local variable in my init.el.
But I think it should be better to do this in this library.

[alphapapa]

That seems like a good idea. Are there any other variables that should also be marked as safe?

[wsw0108]

No other variables needed for me.

[alphapapa]

Why did you close this issue?

[alphapapa]

Since this option is passed to other programs, it should be considered carefully, so deferring until v1.7.

[alphapapa]

I'd like to do this, but I'm still not sure if it's a good idea. Since we call commands with lists of arguments (rather than as a shell command string), it seems like it ought to be safe, but to be sure we'd need to carefully look at each command that is called, how the arguments are built using the values in this variable, and consider whether there's any way that the arguments could be manipulated into something nefarious.

As an analogy, see this discussion about marking a similar variable as safe in Emacs; Eli is being very cautious about it, and we should be as cautious, IMO.

So I'm targeting this at a later version. In the meantime, users can press ! to mark sets of values as safe in their configuration so as to not be prompted anymore, like other variables.