Skip to content

Conversation

@hendisantika
Copy link

This PR includes:

  1. Upgraded into Spring Boot 3.5.3 version πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
  2. Setup GitHub Action πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
  3. Add DependabotπŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†

β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
β€¦πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
… πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
@ved-asole
Copy link
Owner

Checks are failing, please debug

…version πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
…version πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
…version πŸ«ΈπŸŒ€βœοΈπŸ“—πŸ§πŸ³β¬†
@ved-asole ved-asole requested a review from Copilot September 20, 2025 09:06
Copy link

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

This PR upgrades the project from Spring Boot 2.7.18 to Spring Boot 3.5.6 with Java 17 to Java 21, requiring migration from javax to jakarta namespace and updates to deprecated configurations.

  • Upgraded Spring Boot to 3.5.6 and Java to version 21
  • Migrated all imports from javax.* to jakarta.* namespace (mail, servlet, validation, persistence, transaction)
  • Updated Spring Security configuration to use modern lambda-style configuration syntax and replaced deprecated annotations

Reviewed Changes

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

Show a summary per file
File Description
pom.xml Updated Spring Boot version to 3.5.6, Java to 21, replaced Springfox with SpringDoc OpenAPI
Entity classes Migrated JPA annotations from javax.persistence to jakarta.persistence
Payload DTOs Migrated validation annotations from javax.validation to jakarta.validation
Service classes Updated imports for jakarta mail, servlet, validation, and transaction
Security classes Migrated servlet imports to jakarta and updated Spring Security configuration
Controllers Migrated validation imports and expanded wildcard imports to explicit imports
Properties files Updated PostgreSQL dialect and Redis configuration properties
SwaggerConfig Replaced Springfox with SpringDoc OpenAPI v3 configuration
GitHub workflows Added CI workflow and Dependabot configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Owner

@ved-asole ved-asole left a comment

Choose a reason for hiding this comment

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

Builds are failing. Please check

@hendisantika
Copy link
Author

Give me permission so that I can check it @ved-asole

@sonarqubecloud
Copy link

@hendisantika
Copy link
Author

check again @ved-asole

@ved-asole
Copy link
Owner

The workflows are still failing. Redis is a dependency for the application. It will be required in workflow to run the application. Also there are other compile time errors. Please work on this.

@ved-asole ved-asole self-requested a review September 21, 2025 16:47
Copy link
Owner

@ved-asole ved-asole left a comment

Choose a reason for hiding this comment

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

The workflows are still failing. Redis is a dependency for the application. It will be required in workflow to run the application. Also there are other compile time errors. Please work on this.

@hendisantika
Copy link
Author

Give me permission so that I can see the logs before it merge @ved-asole

@hendisantika
Copy link
Author

image

Merge it then we know.

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.

2 participants