| ▲ | ocornut 5 hours ago | |
It baffles me that any maintainer would merge code like the one highlighted in the issue, without knowing what it does. That’s regardless of being or not being able to see the “invisible” characters. There’s a transforming function here and an eval() call. The mere fact that a software maintainer would merge code without knowing what it does says more about the terrible state of software. | ||
| ▲ | pdonis an hour ago | parent | next [-] | |
Wish I could upvote this more. | ||
| ▲ | mmlb 23 minutes ago | parent | prev [-] | |
In this instance the PR that was merged was from 6 years ago and was clear https://github.com/pedronauck/reworm/pull/28. Looks to me like a force push overwrote the commit that now exists in history since it was done 6y later. | ||