triallax
triallax
> Do you have any way to reproduce it? I have no idea unfortunately, I'm still not quite sure what triggered the bug in the first place. > Probably can...
Actually, I could restore from my backup yesterday and see if I can reproduce the panic, I'll try that.
https://github.com/charmbracelet/bubbletea/blob/3e050bbad12f880dbe960129d20663ecd836e775/key.go#L544 I suspect this means that something in the diff/one of my files was triggering the error (supported by the fact that `chezmoi update` fixed it).
Yeah I couldn't reproduce it unfortunately, they I'm sure it's not about the diff length because I had a huge diff at one point and that didn't crash `chezmoi diff`.
Well, if you go a couple lines up: https://github.com/lee-dohm/no-response/blob/1ae4f8bbc8a4607bb904abc1f5f7556ea2ace95b/src/no-response.ts#L130 It filters out PRs.
Haha, fair enough, but I'll keep this open as an issue that requests PR support.
With PureScript 0.14.2 being released with automatic instance naming, I suspect that this is not needed anymore. However, I think 5.x versions of this library should probably be compatible with...
Good point; perhaps we should wait until all bugs related to that have been wrinkled out first.
@garyb you do make a good point. I just think that the discoverability of this is low for new people. Should I add an example to the cookbook?
I'm not sure what to do here. Should I open a PR here and add `startsWith` and `endsWith` like @natefaubion is suggesting, or should I add examples to the cookbook?