Fix Checkstyle issues

aws · Sep 26, 2024 · 5e793f5 · 5e793f5
1 parent c304a7f
commit 5e793f5
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 18 deletions.
diff --git a/plugin/pom.xml b/plugin/pom.xml
@@ -221,16 +221,15 @@
                         <version>10.17.0</version>
                     </dependency>
                 </dependencies>
+                <configuration>
+                    <configLocation>checkstyle.xml</configLocation>
+                    <consoleOutput>true</consoleOutput>
+                    <failsOnError>true</failsOnError>
+                </configuration>
                 <executions>
                     <execution>
                         <id>checkstyle-validation</id>
                         <phase>validate</phase>
-                        <configuration>
-                            <configLocation>checkstyle.xml</configLocation>
-                            <consoleOutput>true</consoleOutput>
-                            <failsOnError>true</failsOnError>
-                            <includes>src/**/*.java</includes>
-                        </configuration>
                         <goals>
                             <goal>check</goal>
                         </goals>

diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/chat/ChatCommunicationManager.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/chat/ChatCommunicationManager.java
@@ -57,4 +57,4 @@ public void sendMessageToChatUI(final Browser browser, final ChatUIInboundComman
         });
     }
 
-}
+}
diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/chat/ChatMessageProvider.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/chat/ChatMessageProvider.java
@@ -22,7 +22,7 @@ public static CompletableFuture<ChatMessageProvider> createAsync() {
                 .thenApply(ChatMessageProvider::new);
     }
 
-    private ChatMessageProvider(AmazonQLspServer amazonQLspServer) {
+    private ChatMessageProvider(final AmazonQLspServer amazonQLspServer) {
         this.amazonQLspServer = amazonQLspServer;
     }
 

diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/util/QInvocationSession.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/util/QInvocationSession.java
@@ -339,8 +339,8 @@ public void executeCallbackForCodeReference() {
             codeReferenceAcceptanceCallback.onCallback(selectedSuggestion, startLine);
         }
     }
-  
-    public void setVerticalIndent(int line, int height) {
+
+    public void setVerticalIndent(final int line, final int height) {
         var widget = viewer.getTextWidget();
         widget.setLineVerticalIndent(line, height);
         unsetVerticalIndent = () -> {

diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/views/AmazonQChatViewActionHandler.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/views/AmazonQChatViewActionHandler.java
@@ -61,4 +61,4 @@ public final void handleCommand(final ParsedCommand parsedCommand, final Browser
         }
     }
 
-}
+}
diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/views/FeedbackDialog.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/views/FeedbackDialog.java
@@ -217,16 +217,16 @@ private void createQuestionSection(final Composite container) {
         CustomRadioButton negativeSentimentButton = createCustomRadioButton(sentimentContainer, "icons/FrownyFace.png", "Unsatisfied", SWT.NONE, false);
         positiveSentimentButton.getRadioButton().addSelectionListener(new SelectionAdapter() {
             @Override
-            public void widgetSelected(SelectionEvent e) {
-            	negativeSentimentButton.getRadioButton().setSelection(false);
-            	selectedSentiment = Sentiment.POSITIVE;
+            public void widgetSelected(final SelectionEvent e) {
+                negativeSentimentButton.getRadioButton().setSelection(false);
+                selectedSentiment = Sentiment.POSITIVE;
             }
         });
         negativeSentimentButton.getRadioButton().addSelectionListener(new SelectionAdapter() {
             @Override
-            public void widgetSelected(SelectionEvent e) {
-            	positiveSentimentButton.getRadioButton().setSelection(false);
-            	selectedSentiment = Sentiment.NEGATIVE;
+            public void widgetSelected(final SelectionEvent e) {
+                positiveSentimentButton.getRadioButton().setSelection(false);
+                selectedSentiment = Sentiment.NEGATIVE;
             }
         });
 
@@ -280,7 +280,8 @@ private void createSeparator(final Composite parent) {
         separatorLabel.setLayoutData(separatorGithubLayout);
     }
 
-    private CustomRadioButton createCustomRadioButton(final Composite parent, final String imagePath, final String text, final int style, final boolean isSelected) {
+    private CustomRadioButton createCustomRadioButton(final Composite parent, final String imagePath,
+            final String text, final int style, final boolean isSelected) {
         CustomRadioButton button = new CustomRadioButton(parent, loadImage(imagePath), text, style);
         button.getRadioButton().setSelection(isSelected);
         return button;
-Original file line number
+Diff line change
@@ Expand Up @@
             });
         }
-    }
+    }