- 
                Notifications
    
You must be signed in to change notification settings  - Fork 273
 
feat(amazonq): allow users to skip tests #4887
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
| 
           Note: backend work for this feature is not done yet, so not planning on merging this PR quite yet  | 
    
7742c09    to
    f9f8fe5      
    Compare
  
    7ec1611    to
    1f28a70      
    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.
Approved pending final string review
        
          
                ...are/aws/toolkits/jetbrains/services/codemodernizer/controller/CodeTransformChatController.kt
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                ...are/aws/toolkits/jetbrains/services/codemodernizer/controller/CodeTransformChatController.kt
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                ...are/aws/toolkits/jetbrains/services/codemodernizer/controller/CodeTransformChatController.kt
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                ...are/aws/toolkits/jetbrains/services/codemodernizer/controller/CodeTransformChatController.kt
              
                Outdated
          
            Show resolved
            Hide resolved
        
      | 
               | 
          ||
| private val transformationStoppedByUsr = AtomicBoolean(false) | ||
| private var codeTransformationSession: CodeModernizerSession? = null | ||
| var codeTransformationSession: CodeModernizerSession? = null | 
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.
see if you can make this private again

Types of changes
Description
Prompt the user with a form asking them if they want us to skip integ tests or skip all tests when doing our server-side build. Pass the flag via the manifest.json.
Checklist
License
I confirm that my contribution is made under the terms of the Apache 2.0 license.