Skip to content

Commit 2927885

Browse files
authored
chore!: remove deprecated "platform-improvements" feedback survey name (#465)
1 parent 5100c02 commit 2927885

2 files changed

Lines changed: 2 additions & 20 deletions

File tree

cmd/feedback/feedback.go

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,8 @@ const (
6868

6969
// Supported survey names
7070
const (
71-
SlackCLIFeedback = "slack-cli"
72-
SlackPlatformFeedback = "slack-platform"
73-
SlackPlatformFeedbackDeprecated = "platform-improvements" // DEPRECATED(semver:major)
71+
SlackCLIFeedback = "slack-cli"
72+
SlackPlatformFeedback = "slack-platform"
7473
)
7574

7675
type SurveyConfigInterface interface {
@@ -241,12 +240,6 @@ func runFeedbackCommand(ctx context.Context, clients *shared.ClientFactory, cmd
241240
return nil
242241
}
243242

244-
// DEPRECATED(semver:major): Support the deprecated survey name for backwards compatibility
245-
if surveyNameFlag == SlackPlatformFeedbackDeprecated {
246-
surveyNameFlag = SlackPlatformFeedback
247-
clients.IO.PrintWarning(ctx, "DEPRECATED: The '--name %s' flag is deprecated; use '--name %s' instead", SlackPlatformFeedbackDeprecated, SlackPlatformFeedback)
248-
}
249-
250243
surveyNames, surveyPromptOptions := initSurveyOpts(ctx, clients, SurveyStore)
251244

252245
if _, ok := SurveyStore[surveyNameFlag]; !ok && surveyNameFlag != "" {

cmd/feedback/feedback_test.go

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -263,17 +263,6 @@ func TestShowSurveyMessages(t *testing.T) {
263263

264264
func Test_Feedback_FeedbackCommand(t *testing.T) {
265265
testutil.TableTestCommand(t, testutil.CommandTests{
266-
// DEPRECATED(semver:major): Support the deprecated survey name for backwards compatibility
267-
"supports deprecated --name platform-improvements": {
268-
CmdArgs: []string{"--name", "platform-improvements"},
269-
Setup: func(t *testing.T, ctx context.Context, cm *shared.ClientsMock, cf *shared.ClientFactory) {
270-
setupFeedbackCommandMocks(t, ctx, cm, cf)
271-
},
272-
ExpectedOutputs: []string{
273-
"feedback@slack.com",
274-
"https://docs.slack.dev/developer-support",
275-
},
276-
},
277266
"supports --name slack-cli": {
278267
CmdArgs: []string{"--name", "slack-cli"},
279268
Setup: func(t *testing.T, ctx context.Context, cm *shared.ClientsMock, cf *shared.ClientFactory) {

0 commit comments

Comments
 (0)