Skip to content

Conversation

@nkomonen-amazon
Copy link
Contributor

@nkomonen-amazon nkomonen-amazon commented Nov 27, 2024

Problem

We didn't have much use for the old "Amazon Q" channel in addition to "Amazon Q Logs". All the logs we used were in "Amazon Q Logs".

Solution

  • Now, just have the useful "Amazon Q Logs" channel.
  • Move the Amazon Q Language Server logs in to Amazon Q Logs to unify all logs

License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@nkomonen-amazon nkomonen-amazon requested a review from a team as a code owner November 27, 2024 19:15
@github-actions
Copy link

  • This pull request modifies code in src/* but no tests were added/updated.
    • Confirm whether tests should be added or ensure the PR description explains why tests are not required.

Problem

We didn't have much use for the old "Amazon Q" channel in addition to
"Amazon Q Logs". All the logs we used were in "Amazon Q Logs".

Solution

Now, just have the useful "Amazon Q Logs" channel.

Signed-off-by: nkomonen-amazon <[email protected]>
Signed-off-by: nkomonen-amazon <[email protected]>
We want to have everything in one place.

Signed-off-by: nkomonen-amazon <[email protected]>
@nkomonen-amazon nkomonen-amazon requested a review from a team as a code owner November 27, 2024 22:16
},
// Log to the Amazon Q Logs so everything is in a single channel
// TODO: Add prefix to the language server logs so it is easier to search
outputChannel: globals.logOutputChannel,
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice!

@nkomonen-amazon nkomonen-amazon changed the base branch from master to feature/postreinvent December 2, 2024 20:18
@nkomonen-amazon
Copy link
Contributor Author

/retryBuilds

@nkomonen-amazon nkomonen-amazon merged commit 6d5e016 into aws:feature/postreinvent Dec 3, 2024
42 of 45 checks passed
@nkomonen-amazon nkomonen-amazon deleted the removeQOutputChannel branch December 3, 2024 19:56
karanA-aws pushed a commit to karanA-aws/aws-toolkit-vscode that referenced this pull request Jan 17, 2025
…aws#6114)

## Problem

We didn't have much use for the old "Amazon Q" channel in addition to
"Amazon Q Logs". All the logs we used were in "Amazon Q Logs".

## Solution

- Now, just have the useful "Amazon Q Logs" channel.
- Move the Amazon Q Language Server logs in to Amazon Q Logs to unify
all logs


---

<!--- REMINDER: Ensure that your PR meets the guidelines in
CONTRIBUTING.md -->

License: I confirm that my contribution is made under the terms of the
Apache 2.0 license.

---------

Signed-off-by: nkomonen-amazon <[email protected]>
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