Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

remove beta labels for code monitoring Slack notifs & logs#63480

Merged
sqs merged 1 commit into
mainfrom
sqs/rm-codemonitor-beta-labels
Jun 26, 2024
Merged

remove beta labels for code monitoring Slack notifs & logs#63480
sqs merged 1 commit into
mainfrom
sqs/rm-codemonitor-beta-labels

Conversation

@sqs

@sqs sqs commented Jun 25, 2024

Copy link
Copy Markdown
Member

These features have passed the steps needed to take them out of beta. They have been feature-complete and in widespread use for more than a year with no outstanding issues, and they work well.

Closes https://linear.app/sourcegraph/issue/SRCH-533/remove-beta-label-from-code-monitor-logs-tab

Test plan

Create a new code monitor with a Slack notif. Ensure it works e2e by pushing a commit that matches and watching it show up in Slack (https://sourcegraph.slack.com/archives/C68386Z5Z/p1719357942294309).

Changelog

  • Code monitoring Slack notifications, previously in beta, are now GA. This feature lets you post matching code changes to a Slack channel.
  • Code monitoring logs, previously in beta, are now GA. This feature lets you see the status of and activity related to code monitors to troubleshoot issues with finding results and performing configured actions.

@sqs sqs requested a review from a team June 25, 2024 23:31
@cla-bot cla-bot Bot added the cla-signed label Jun 25, 2024

@FabianaCampanari FabianaCampanari left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update

These features have passed the steps needed to take them out of beta. They have been feature-complete and in widespread use for more than a year with no outstanding issues, and they work well.

- https://linear.app/sourcegraph/project/slack-notifications-ga-code-monitoring-7dbb9fa13f62
- https://linear.app/sourcegraph/issue/SRCH-533/remove-beta-label-from-code-monitor-logs-tab
@sqs sqs force-pushed the sqs/rm-codemonitor-beta-labels branch from c832636 to cba2308 Compare June 25, 2024 23:50

@peterguy peterguy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM!

@sqs sqs merged commit b79cf5b into main Jun 26, 2024
@sqs sqs deleted the sqs/rm-codemonitor-beta-labels branch June 26, 2024 00:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants