Skip to content

Conversation

@ntarakad-aws
Copy link
Contributor

Types of changes

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

Description

Selective transformation changes were released under an IDE feature flag due to backend changes that needed to be fixed. In this PR, we are removing the feature flag on the IDE side so the users will now be able to see the form to select if they want to view their changes in one or multiple diffs. If they select multiple diffs in this form, then selective transformation will be invoked.

License

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

@ntarakad-aws ntarakad-aws self-assigned this Nov 21, 2024
@ntarakad-aws ntarakad-aws requested review from a team as code owners November 21, 2024 21:44
@dhasani23
Copy link
Contributor

Please confirm with @ntarakad-aws that this should be merged before merging

@ntarakad-aws ntarakad-aws force-pushed the selective-transform-release branch from 9bd69bd to 20f67ee Compare November 22, 2024 05:56
@dhasani23
Copy link
Contributor

Still waiting, will let you know if this should be merged

@dhasani23
Copy link
Contributor

Got a SEV-2, so this will go out next week

@dhasani23 dhasani23 marked this pull request as draft November 22, 2024 18:49
@dhasani23 dhasani23 marked this pull request as ready for review November 25, 2024 17:53
@dhasani23 dhasani23 merged commit 91d5a0c into aws:main Nov 25, 2024
11 checks passed
karanA-aws pushed a commit to karanA-aws/aws-toolkit-jetbrains that referenced this pull request Jan 22, 2025
…d confirmed string changes (aws#5117)

* removing feature flag for selective transformation and confirmed string changes

* fix to use refreshAndFindFileByNioFile instead

---------

Co-authored-by: David <[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.

3 participants