Skip to content

Commit d82aa7a

Browse files
ToreMerkelyclaude
andcommitted
fix: correct the scaling deprecation message
The old wording claimed scaling events no longer create environment snapshots, which is not true for the grandfathered environments that still capture scaling. State the plain fact instead: scaling events do not trigger new snapshots. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent 27c88bc commit d82aa7a

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

‎cmd/kosli/createEnvironment.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ func newCreateEnvironmentCmd(out io.Writer) *cobra.Command {
9797
cmd.Flags().BoolVar(&o.requireProvenance, "require-provenance", false, requireProvenanceFlag)
9898
cmd.Flags().StringSliceVar(&o.payload.IncludedEnvironments, "included-environments", []string{}, includedEnvironments)
9999

100-
scalingDeprecationMsg := "this flag is deprecated and will be removed in a future version. Scaling events no longer create environment snapshots."
100+
scalingDeprecationMsg := "this flag is deprecated and will be removed in a future version. Scaling events do not trigger new snapshots."
101101
err := DeprecateFlags(cmd, map[string]string{
102102
"require-provenance": "this flag is deprecated and will be removed in a future version. Use policies instead.",
103103
"include-scaling": scalingDeprecationMsg,

‎cmd/kosli/snapshotAutoEnvironment.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ func addAutoEnvironmentFlags(cmd *cobra.Command) {
3939
cmd.PersistentFlags().BoolVar(&snapshotAutoEnv.includeScaling, "include-scaling", false, includeScalingFlag)
4040
cmd.PersistentFlags().BoolVar(&snapshotAutoEnv.excludeScaling, "exclude-scaling", false, excludeScalingFlag)
4141

42-
scalingDeprecationMsg := "this flag is deprecated and will be removed in a future version. Scaling events no longer create environment snapshots."
42+
scalingDeprecationMsg := "this flag is deprecated and will be removed in a future version. Scaling events do not trigger new snapshots."
4343
for _, name := range []string{"include-scaling", "exclude-scaling"} {
4444
if err := cmd.PersistentFlags().MarkDeprecated(name, scalingDeprecationMsg); err != nil {
4545
logger.Error("failed to mark %s as deprecated: %v", name, err)

0 commit comments

Comments
 (0)