8000 [WINDUPRULE-941] Migration from hibernate-search 6.0 to 6.1 by jmle · Pull Request #844 · windup/windup-rulesets · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[WINDUPRULE-941] Migration from hibernate-search 6.0 to 6.1 #844

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

Merged
merged 7 commits into from
Feb 16, 2023

Conversation

jmle
Copy link
Contributor
@jmle jmle commented Jan 23, 2023

@PhilipCattanach
Copy link
Contributor

Thanks @jmle - I have reviewed and tested the PR and overall it is good.
One minor change is required to the rule # 20 the word to is repeated within the highlighted text.
to `to
The thing that is missing are the rules for deprecated API's
We always create rules under the Migration Optional category to warn users of Deprecated APIs that will be deleted in a future release.

@PhilipCattanach
Copy link
Contributor

Hi Juanma,
When I was looking through the rules folders I noticed that we have some old hibernate rules in the hibernate rules folder.
However for more recent releases of EAP we have tended to put the Hibernate rules in the eap rules folders. For example eap7/eap71
Could I ask for 1 final change please and move this ruleset into the eap8/eap7 folder please?
Thanks

@PhilipCattanach
Copy link
Contributor

Hi @jmle
Rule hibernate-search-6.1-00020 still has the superfluous 'to' within it.
Rule hibernate-search-6.1-00040 please change the message from 'now API' to 'now an API'
Rules hibernate-search-6.1-00150 and hibernate-search-6.1-00160 I think a , rather than a - would read better
Rules hibernate-search-6.1-00170 and hibernate-search-6.1-00180 I think a , rather than a : would read better
Thanks,
Phil

@PhilipCattanach
Copy link
Contributor

Thanks @jmle - All good. Just waiting for the automated tests to pass then I'll merge the Pr and update the corresponding Jira.

@PhilipCattanach PhilipCattanach merged commit e4c583e into windup:master Feb 16, 2023
@mrizzi mrizzi added auto-backport Set the PR for being backported auto-backport-to-releases/6.1.z Backport this issue to releases/6.1.z branch labels Mar 10, 2023
github-actions bot pushed a commit that referenced this pull request Mar 10, 2023
* [WINDUPRULE-941] Migration from hibernate-search 6.0 to 6.1

* [WINDUPRULE-941] Add rules for deprecated usages

* WINDUPRULE-941 add link for rules related to deprecated API

* [WINDUPRULE-941] Move hibernate rules to eap migration path

* [WINDUPRULE-941] Refactor test artifact info

* [WINDUPRULE-941] Fix typos

* WINDUPRULE-941 minor message punctuation changes to 2 rules related to deprecated components for consistency

---------

Co-authored-by: PhilipCattanach <pcattana@redhat.com>
(cherry picked from commit e4c583e)
@github-actions
Copy link

💚 All backports created successfully

Status Branch Result
releases/6.1.z

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

mrizzi pushed a commit that referenced this pull request Mar 10, 2023
)

* [WINDUPRULE-941] Migration from hibernate-search 6.0 to 6.1

* [WINDUPRULE-941] Add rules for deprecated usages

* WINDUPRULE-941 add link for rules related to deprecated API

* [WINDUPRULE-941] Move hibernate rules to eap migration path

* [WINDUPRULE-941] Refactor test artifact info

* [WINDUPRULE-941] Fix typos

* WINDUPRULE-941 minor message punctuation changes to 2 rules related to deprecated components for consistency

---------

Co-authored-by: PhilipCattanach <pcattana@redhat.com>
(cherry picked from commit e4c583e)

Co-authored-by: Juan Manuel Leflet Estrada <jleflete@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Set the PR for being backported auto-backport-to-releases/6.1.z Backport this issue to releases/6.1.z branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0