Skip to content

Conversation

@patvid
Copy link
Contributor

@patvid patvid commented Mar 28, 2025

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

  1. Updated the service-2.json file of the codewhispererstreaming client, copied the output from /aws-sdk-external-2023-11-27/c2j/codewhispererstreaming-2023-11-27.normal.json.
  2. Certain build (./gradlew :plugin-amazonq:buildPlugin) and test case (./gradlew :plugin-amazonq:test) were failing as a result also applied small fixes for it in files:
  • plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/cwc/clients/chat/v1/ChatSessionV1.kt
  • plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/cwc/controller/chat/telemetry/TelemetryHelper.kt
  • plugins/amazonq/codetransform/jetbrains-community/src/software/aws/toolkits/jetbrains/services/codemodernizer/ArtifactHandler.kt
  • plugins/amazonq/codetransform/jetbrains-community/src/software/aws/toolkits/jetbrains/services/codemodernizer/constants/CodeTransformChatItems.kt

Note: Not completely sure if the values used in the files in step2 are correct, would appreciate if the respective team can review and confirm the changes.

Checklist

  • My code follows the code style of this project
  • I have added tests to cover my changes
  • A short description of the change has been added to the CHANGELOG if the change is customer-facing in the IDE.
  • I have added metrics for my changes (if required)

License

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

@patvid patvid requested review from a team as code owners March 28, 2025 21:54
@patvid patvid force-pushed the model-update branch 3 times, most recently from 62b0944 to b1b5477 Compare April 7, 2025 18:15
@patvid patvid force-pushed the model-update branch 2 times, most recently from 4c4272a to 13e6f9e Compare April 10, 2025 15:57
@rli rli merged commit 5f35e4d into aws:main Apr 11, 2025
13 of 14 checks passed
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.

3 participants