Skip to content

AUT-7054 update all deps, bump version, fix linting#49

Merged
noobgramming merged 1 commit intomasterfrom
AUT-7054
Jul 8, 2025
Merged

AUT-7054 update all deps, bump version, fix linting#49
noobgramming merged 1 commit intomasterfrom
AUT-7054

Conversation

@noobgramming
Copy link
Copy Markdown
Contributor

What changed?

  • Update all dependencies
  • Fix linter issues

@noobgramming noobgramming requested a review from Copilot July 7, 2025 20:37
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Update project version to 6.1.0-SNAPSHOT, bump dozens of dependencies to their latest compatible releases, and clean up linting configuration and suppressions across modules.

  • Bump parent and module versions, plus a large matrix of dependencies and plugin versions in all pom.xml files
  • Refine SpotBugs and PMD rulesets (add new excludes, remove deprecated rules)
  • Remove or add @SuppressWarnings and @FunctionalInterface annotations to satisfy lint rules, and adjust minor code behavior (Gson, charsets, enum logic)

Reviewed Changes

Copilot reviewed 51 out of 51 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pom.xml Bumped project version and updated all dependency & plugin versions
build-tools/spotbugs-exclude.xml Added exclusion for RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
build-tools/pmdconfig.xml Removed many deprecated rules, reorganized exclusions
auto-sdk-java-framework/src/main/java/.../AbstractDriverManager.java Removed .setLenient() on GsonBuilder
auto-sdk-java-common/src/main/java/.../SwipeDirection.java Removed default exception branch in enum switch
Comments suppressed due to low confidence (4)

auto-sdk-java-common/src/main/java/com/applause/auto/data/enums/SwipeDirection.java:60

  • The default exception case was removed, so unhandled enum values will now fall through silently. Consider restoring the default branch or explicitly handling invalid values to prevent unexpected behavior.
            start = new Point((int) (width * 0.15), height / 2);

auto-sdk-java-framework/src/main/java/com/applause/auto/framework/AbstractDriverManager.java:44

  • Removed .setLenient() changes JSON parsing behavior and may break lenient parsing use-cases. If lenient mode is required, re-add .setLenient() or verify that all JSON inputs conform to strict parsing.
  protected static final Gson gsonLenient = new GsonBuilder().create();

auto-sdk-java-config/src/main/java/com/applause/auto/config/PropertyHelper.java:204

  • [nitpick] Removing the PMD.CognitiveComplexity suppression may surface complexity warnings here. Consider refactoring this method to reduce complexity or reintroducing a targeted suppression with justification.
  static <T extends Config> String callForValueAsString(

auto-sdk-java-testng/src/main/java/com/applause/auto/testng/dataprovider/TestIterationTag.java:25

  • [nitpick] Suppressing PMD.AvoidFieldNameMatchingTypeName was removed, which could trigger linter warnings since the field name matches the record name. Consider reintroducing the suppression or renaming the parameter for clarity.
public record TestIterationTag(String testIterationTag) implements ITestIterationTag {

@noobgramming noobgramming merged commit d394a57 into master Jul 8, 2025
6 checks passed
@noobgramming noobgramming deleted the AUT-7054 branch July 8, 2025 14:15
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