ну да, по отношению к этой функции глобальная, смысл ясен, но это получается более замысловато.. глобальные переменные плохи ведь тем, что их можно перезаписать, верно по неосторожности?! И поэтому нужно сводить это дело к минимуму.
В данном случае эту переменную нужно перезаписывать, но сейчас она у меня в явном виде перезаписывается, а если я ее передам параметром в дальнейшем, в любом случае она точно также перезапишется, но тогда появится больше кода, потому что в запросе идет сам запрос сначала, а потом вызов этой ф-и:

   clearResults(results); 
   checkData(jsonGetData); // Тогда сюда еще нужно будет передавать этот results 2ым
   getRepos(login); 

или весь смысл в том, чтобы из принципа не использовать глобальные?!)