self.urls is not deterministic between OSs, so on Windows it returns remote repositories last, and on macOS it returns them first. This commit makes it so we don't rely on self.urls.
Also, since the for loop doesn't access or use url variable, I've removed it.
Closes: #372
Code review checklist (for code reviewer to complete)
[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[ ] Title summarizes what is changing
[ ] Commit messages are meaningful (see this for details)
[ ] Tests have been included and/or updated, as appropriate
[ ] Docstrings have been included and/or updated, as appropriate
[ ] Changelog has been updated, as needed (see CHANGELOG.md)
Description (e.g. "Related to ...", etc.)
self.urls
is not deterministic between OSs, so on Windows it returns remote repositories last, and on macOS it returns them first. This commit makes it so we don't rely onself.urls
.Also, since the for loop doesn't access or use url variable, I've removed it.
Closes: #372
Code review checklist (for code reviewer to complete)