Тернарный оператор не нужен для потока управления и более полезен, если вы пытаетесь вернуть значение из оператора. Вам следует подумать о том, чтобы упростить его до простого оператора if
.
if (scores[activePlayer] !== 20)
nextPlayer();
и просто позволить функции выйти естественным образом, если false.