Skip to content

Conversation

@justinmk3
Copy link
Contributor

Problem

The vscode Output panel now supports filtering:
https://code.visualstudio.com/updates/v1_97#_output-panel-filtering
This means we no longer need the AWS (Developer): Watch Logs command.

Solution

Remove the AWS (Developer): Watch Logs command.

ref #4176


  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

Problem:
The vscode Output panel now supports filtering:
https://code.visualstudio.com/updates/v1_97#_output-panel-filtering
This means we no longer need the `AWS (Developer): Watch Logs` command.

Solution:
Remove the `AWS (Developer): Watch Logs` command.

ref aws#4176
@justinmk3 justinmk3 requested a review from a team as a code owner February 7, 2025 21:06
@github-actions

This comment was marked as resolved.

@justinmk3 justinmk3 changed the title refactor(logs)!: drop refactor(logs)!: remove "Watch Logs" command Feb 7, 2025
@justinmk3 justinmk3 changed the title refactor(logs)!: remove "Watch Logs" command refactor(logs): remove "Watch Logs" command Feb 7, 2025
@justinmk3 justinmk3 closed this Feb 7, 2025
@justinmk3 justinmk3 reopened this Feb 7, 2025
@justinmk3
Copy link
Contributor Author

/runintegrationtests

@justinmk3 justinmk3 merged commit 53f11e7 into aws:master Feb 10, 2025
25 of 30 checks passed
@justinmk3 justinmk3 deleted the droptelemwatch branch February 10, 2025 13:58
s7ab059789 pushed a commit to s7ab059789/aws-toolkit-vscode that referenced this pull request Feb 19, 2025
## Problem
The vscode Output panel now supports filtering:
https://code.visualstudio.com/updates/v1_97#_output-panel-filtering 
This means we no longer need the `AWS (Developer): Watch Logs` command.

## Solution
Remove the `AWS (Developer): Watch Logs` command.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants