-
Notifications
You must be signed in to change notification settings - Fork 201
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
feat(codewhisperer): make CWSPR connection application wide #3658
Conversation
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.
A E2E flow showing CW connection is stored in the application-scope aws.xml can be attached in the PR.
Also if there are not tests for ToolkitConnectionManager application services, we should also add to this PR.
...tware/aws/toolkits/jetbrains/services/codewhisperer/codescan/CodeWhispererCodeScanManager.kt
Outdated
Show resolved
Hide resolved
...ns-core/src/software/aws/toolkits/jetbrains/services/codewhisperer/util/CodeWhispererUtil.kt
Outdated
Show resolved
Hide resolved
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.
this looks good enough.
Just to make sure, could you also verify that the states are persisted in
~/Library/Application Support/JetBrains/<your ide version>/options/aws.xml
?
...ns-core/src/software/aws/toolkits/jetbrains/services/codewhisperer/util/CodeWhispererUtil.kt
Outdated
Show resolved
Hide resolved
yes updated a video for this in the PR description |
...core/src/software/aws/toolkits/jetbrains/core/credentials/DefaultToolkitConnectionManager.kt
Outdated
Show resolved
Hide resolved
...core/src/software/aws/toolkits/jetbrains/core/credentials/DefaultToolkitConnectionManager.kt
Outdated
Show resolved
Hide resolved
...ore/src/software/aws/toolkits/jetbrains/core/credentials/pinning/ConnectionPinningManager.kt
Outdated
Show resolved
Hide resolved
...core/src/software/aws/toolkits/jetbrains/core/credentials/DefaultToolkitConnectionManager.kt
Outdated
Show resolved
Hide resolved
...ains-core/src/software/aws/toolkits/jetbrains/core/credentials/ToolkitAddConnectionDialog.kt
Outdated
Show resolved
Hide resolved
...core/src/software/aws/toolkits/jetbrains/core/credentials/DefaultToolkitConnectionManager.kt
Outdated
Show resolved
Hide resolved
...ore/src/software/aws/toolkits/jetbrains/core/credentials/pinning/ConnectionPinningManager.kt
Outdated
Show resolved
Hide resolved
Kudos, SonarCloud Quality Gate passed!
|
Types of changes
Description
moved to #3671
Screen.Recording.2023-05-12.at.2.21.42.PM.mov
Checklist
License
I confirm that my contribution is made under the terms of the Apache 2.0 license.