Skip to content

Actually skip unsupported repos if -allow-unsupported is not set#403

Merged
mrnugget merged 2 commits into
mainfrom
mrn/fix-unsupported
Dec 11, 2020
Merged

Actually skip unsupported repos if -allow-unsupported is not set#403
mrnugget merged 2 commits into
mainfrom
mrn/fix-unsupported

Conversation

@mrnugget

Copy link
Copy Markdown
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

@eseliger eseliger left a comment

Copy link
Copy Markdown
Member

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.

2 participants