Skip to content

BREAKING: Default SCMs to exclude archived repositories #24019

@agaudreault

Description

@agaudreault

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 ExcludeArchivedRepos should be removed and replaced by IncludeArchivedRepos with default to false.
  • calls to list SCMs repositories should be parametrized to exclude repositories whenever possible
  • breaking change documented

Metadata

Metadata

Assignees

No one assigned

    Labels

    breaking/lowA possibly breaking change with low impactcomponent:application-setsIssue related to the ApplicationSet controllercomponent:appset/generatorIssue related to an Application Set generatorenhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions