Drop pathspec dependency?

[JukkaL]

It looks like pathspec has had two non-backward compatible changes in a fairly small time window which broke CI in master. Since it doesn't seem to provide any super critical functionality, maybe it's less trouble to switch to a custom replacement? Maybe this would even be meaningfully faster in some respects (e.g. import time).

Thoughts?

cc @ilevkivskyi

[ilevkivskyi]

No opinion either way. I vaguely remember @hauntsaninja wanted to write our own implementation. If yes, this may be a good time to do this.

[hauntsaninja]

Yes, I have my own implementation we could use. Are we looking to do this for 2.0 or just in general?

[ilevkivskyi]

Are we looking to do this for 2.0 or just in general?

I think there is no hurry. Since this is not a breaking change, we can do it whenever we want.