-
Notifications
You must be signed in to change notification settings - Fork 627
config(amazonq): disable auto-review by default #7058
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
ecdd254
to
4be8467
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there a particular reason why this is disabled by default? Is there a way for users to re-enable it? Might be worth highlighting those in the github description
@@ -0,0 +1,4 @@ | |||
{ | |||
"type": "Removal", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the type correct? We are not removing the feature but just change the setting?
@@ -0,0 +1,4 @@ | |||
{ | |||
"type": "Removal", | |||
"description": "disable auto-review by default" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we have /review:
in description?
…9a8829-efa0-463b-ba2f-c89ff1cb69e4.json
Problem
We heard from users that auto scans are sometimes causing highlight linting issues where entire IDE is now yellow-underlined with warnings that may sometimes be irrelevant
Solution
We will disable auto scans by default, and user can re-enable in the menu.
feature/x
branches will not be squash-merged at release time.