Skip to content

Update enable field to enabled for automatedSync policy #22439

@anandrkskd

Description

@anandrkskd

Is it too late to change the name of the flag from enable to enabled? It is conventional to name such a flag enabled. For example, there are countless of examples of attributes called .enabled in https://github.com/argoproj/argo-cd/blob/master/manifests/install.yaml, but not a single one called .enable. The Go-code also calls this flag Is...Enabled everywhere, so this is bound to cause confusion later.

Originally posted by @ChristianCiach in #21999 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions