Skip to content
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

147 deprecated exceptions - WIP #202

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

dvaseekara
Copy link
Contributor

No description provided.

@dvaseekara dvaseekara changed the title 147 deprecated exceptions 147 deprecated exceptions - WIP Dec 16, 2024
Copy link
Contributor

@jmle jmle left a comment

Choose a reason for hiding this comment

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

The conditions are correct, but there are a few bits and pieces that need fixing 👍

@@ -90,3 +90,39 @@
url: https://eclipse.dev/eclipselink/downloads/previous_releases.php
- title: 'Spring 6.0 migration guide'
url: https://github.com/spring-projects/spring-framework/wiki/Spring-Framework-6.0-Release-Notes#removed-apis

- ruleID: spring-framework-5.x-to-6.0-data-access-00030
category: mandatory
Copy link
Contributor

Choose a reason for hiding this comment

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

This would be an optional issue instead of a mandatory one, since these members are only deprecated, not removed, so there is no real need to change them yet.

effort: 1
labels:
- konveyor.io/source=springboot2
- konveyor.io/target=springboot3
Copy link
Contributor

Choose a reason for hiding this comment

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

The targets would be:

  labels:
    - konveyor.io/source=spring5
    - konveyor.io/target=spring6+

We'll have to add the spring boot targets eventually, but for now lets just use these.

- konveyor.io/source=springboot2
- konveyor.io/target=springboot3
tags:
- springframework.dao
Copy link
Contributor

Choose a reason for hiding this comment

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

Here, the correct field is tag:, and it would look something like this:

tag:
  - Spring Framework

You can take a look at other rules to see how it's usually done

java.referenced:
pattern: org.springframework.dao.DeadlockLoserDataAccessException
location: IMPORT
description: DeadlockLoserDataAccessException is depracated in favor of CannotAcquireLockException
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo in "deprecated", here and in the other places.

message: "DeadlockLoserDataAccessException is depracated in favor of CannotAcquireLockException"
links:
- title: 'org.springframework.http.HttpMethod Documentation'
url: https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/http/HttpMethod.html
Copy link
Contributor

Choose a reason for hiding this comment

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

This documentation doesn't correspond with the issue

- title: 'org.springframework.http.HttpMethod Documentation'
url: https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/http/HttpMethod.html

- ruleID: spring-framework-5.x-to-6.0-data-access-00040
Copy link
Contributor

Choose a reason for hiding this comment

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

The same comments from the other rule apply here

@@ -0,0 +1,51 @@
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
Copy link
Contributor

Choose a reason for hiding this comment

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

Lets use the existing project in data/data-access - basically move this project up one step and merge the poms. We just need to make sure that all the tests for data-access still pass.

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