Skip to content

DMP-5613 Update Multiple Dependencies Flagged By Renovate#3355

Open
cgi-jhjsimpson wants to merge 2 commits into
masterfrom
DMP-5613-update-multiple-dependencies-flagged-by-renovate
Open

DMP-5613 Update Multiple Dependencies Flagged By Renovate#3355
cgi-jhjsimpson wants to merge 2 commits into
masterfrom
DMP-5613-update-multiple-dependencies-flagged-by-renovate

Conversation

@cgi-jhjsimpson

@cgi-jhjsimpson cgi-jhjsimpson commented Jul 9, 2026

Copy link
Copy Markdown

JIRA link

DMP-5613

Change description

  • Update plugin io.freefair.lombok to v8.14.4
  • Update plugin org.openapi.generator to v7.23.0
  • Update dependency org.springdoc:springdoc-openapi-starter-webmvc-ui to v2.8.17
  • Update dependency com.atlassian.oai:swagger-request-validator-core to v3
  • Update dependency org.junit.jupiter:junit-jupiter to v6
  • Update dependency org.springframework:spring-webflux to v7
  • Update dependency org.springframework.data:spring-data-envers to v4
  • Update dependency uk.gov.service.notify:notifications-java-client to v6

Fixes were made to a couple of test files, and build.gradle.

  • AI-assisted

AI Specific Change description

Assistance on pinning down some of the more complex conflicts and error messaging.

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

@cgi-jhjsimpson cgi-jhjsimpson requested a review from a team as a code owner July 9, 2026 14:11
@cgi-jhjsimpson cgi-jhjsimpson requested review from karen-hedges, npv0 and stephenokeefe and removed request for a team July 9, 2026 14:11
private boolean isQuickSearch;
@JsonProperty("isUsedForRM")
private boolean isUsedForRm;
private boolean usedForRm;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

don't change this as this is a response from ARM so we can't rename. Add a suppression for this if that will fix the issue

@cgi-jhjsimpson cgi-jhjsimpson Jul 10, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This is solved by manually setting the getter and setter:

        // These tests read our OpenAPI YAML directly and do not need springdoc.
        // Exclude springdoc's Swagger jars from the test runtime to avoid duplicate Swagger classes.
        @Getter(onMethod_ = @JsonProperty("isUsedForRM"))
        @Setter(onMethod_ = @JsonProperty("isUsedForRM"))
        private boolean isUsedForRm;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants