Skip to content

Conversation

@mrnugget
Copy link
Contributor

I think multiple changes to this method lead to a regression where
unsupported repositories were correctly being reported as unsupported
but were still added to the final list of repositories, regardless of
whether -allow-unsupported was set or not.

I think multiple changes to this method lead to a regression where
unsupported repositories were correctly being reported as unsupported
but were still added to the final list of repositories, regardless of
whether `-allow-unsupported` was set or not.
@mrnugget mrnugget requested a review from a team December 11, 2020 12:01
Copy link
Member

@eseliger eseliger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice finally a test for this!

@mrnugget mrnugget merged commit 10672e8 into main Dec 11, 2020
@mrnugget mrnugget deleted the mrn/fix-unsupported branch December 11, 2020 13:42
scjohns pushed a commit that referenced this pull request Apr 24, 2023
* Actually skip unsupported repos if -allow-unsupported is not set

I think multiple changes to this method lead to a regression where
unsupported repositories were correctly being reported as unsupported
but were still added to the final list of repositories, regardless of
whether `-allow-unsupported` was set or not.

* Add PR URL to changelog entry
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants