-
Notifications
You must be signed in to change notification settings - Fork 6.9k
Open
Labels
breaking/lowA possibly breaking change with low impactA possibly breaking change with low impactcomponent:application-setsIssue related to the ApplicationSet controllerIssue related to the ApplicationSet controllercomponent:appset/generatorIssue related to an Application Set generatorIssue related to an Application Set generatorenhancementNew feature or requestNew feature or request
Milestone
Description
Summary
After #21505, there was a property added to sometime include or exclude archived repositories. It is very unlikely that people want Archived repositories... The behaviour of different SCMs should be consistent by default.
Motivation
By default, do not query archived repositories for any SCM. If it is possible, the list should be parametrized so archived repositories are not even sent over the wire.
Proposal
- property
ExcludeArchivedReposshould be removed and replaced byIncludeArchivedReposwith default to false. - calls to list SCMs repositories should be parametrized to exclude repositories whenever possible
- breaking change documented
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
breaking/lowA possibly breaking change with low impactA possibly breaking change with low impactcomponent:application-setsIssue related to the ApplicationSet controllerIssue related to the ApplicationSet controllercomponent:appset/generatorIssue related to an Application Set generatorIssue related to an Application Set generatorenhancementNew feature or requestNew feature or request