respel
respel
As a user, I would be fine with either. From a dev perspective, I would say no because I don't see enough benefits to warrant a special case.
Should we because we don't indent other binary operators such as && or ||?
One heuristic that prettier uses is that if there is only type parameter, it isn't broken up, ever. Maybe, we can incorporate that. More specifically, prettier keeps the LHS part...
Both look OK to me visually. Also, prettier uses the latter. Do you have a concrete case in mind where we can benefit from using the former?
@belav Interesting articles. There's one small catch though. In the links you gave, they are only talking about arithmetic and boolean operators, not assignment. Same goes for `black`.
Hi, I was testing this PR and it seems like the original issue of the engine not passing the sprt test still exists. Relevant ticket https://github.com/cutechess/cutechess/issues/636
@kiudee I see. That was the issue. Thanks for helping out!
@sandermvdb Thanks for the reply, it did help a lot. You are already using only one array for the transpositions table, so that's a non-issue 😄 Regarding the struct itself,...
@sandermvdb Thanks for the tip. Merged move with flag.