Skip to content

Upgrade Hibernate ORM to to 7.0.0.CR2 #2253

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 1 commit into from
May 14, 2025

Conversation

gavinking
Copy link
Member

@gavinking gavinking commented May 14, 2025

adapt to changes to setParameter() signature

Fix #2251

adapt to changes to setParameter() signature
@gavinking gavinking changed the title update to 7.0.0.CR2 [#2251] update to 7.0.0.CR2 May 14, 2025
@DavideD DavideD changed the title [#2251] update to 7.0.0.CR2 Upgrade Hibernate ORM to to 7.0.0.CR2 May 14, 2025
@DavideD
Copy link
Member

DavideD commented May 14, 2025

Could you also add the issue to the commit message, please?

*/
<P> ReactiveQuery<R> setParameter(int parameter, P argument, BindableType<P> type);
<P> ReactiveQuery<R> setParameter(int parameter, P argument, Type<P> type);

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
ReactiveSelectionQuery.setParameter
; it is advisable to add an Override annotation.
This method overrides
ReactiveMutationQuery.setParameter
; it is advisable to add an Override annotation.
@@ -128,7 +128,7 @@

<P> ReactiveQuery<R> setParameter(QueryParameter<P> parameter, P argument, Class<P> type);

<P> ReactiveQuery<R> setParameter(QueryParameter<P> parameter, P argument, BindableType<P> type);
<P> ReactiveQuery<R> setParameter(QueryParameter<P> parameter, P argument, Type<P> type);

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
ReactiveSelectionQuery.setParameter
; it is advisable to add an Override annotation.
This method overrides
ReactiveMutationQuery.setParameter
; it is advisable to add an Override annotation.
*
* @apiNote This is used for binding a list of values to an expression
* such as {@code entity.field in (:values)}.
*
* @return {@code this}, for method chaining
*/
<P> ReactiveQuery<R> setParameterList(String parameter, Collection<? extends P> arguments, BindableType<P> type);
<P> ReactiveQuery<R> setParameterList(String parameter, Collection<? extends P> arguments, Type<P> type);

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
ReactiveSelectionQuery.setParameterList
; it is advisable to add an Override annotation.
This method overrides
ReactiveMutationQuery.setParameterList
; it is advisable to add an Override annotation.
*
* @apiNote This is used for binding a list of values to an expression
* such as {@code entity.field in (:values)}.
*
* @return {@code this}, for method chaining
*/
<P> ReactiveQuery<R> setParameterList(String parameter, P[] arguments, BindableType<P> type);
<P> ReactiveQuery<R> setParameterList(String parameter, P[] arguments, Type<P> type);

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
ReactiveSelectionQuery.setParameterList
; it is advisable to add an Override annotation.
This method overrides
ReactiveMutationQuery.setParameterList
; it is advisable to add an Override annotation.
*
* @apiNote This is used for binding a list of values to an expression
* such as {@code entity.field in (:values)}.
*
* @return {@code this}, for method chaining
*/
<P> ReactiveQuery<R> setParameterList(int parameter, Collection<? extends P> arguments, BindableType<P> type);
<P> ReactiveQuery<R> setParameterList(int parameter, Collection<? extends P> arguments, Type<P> type);

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
ReactiveSelectionQuery.setParameterList
; it is advisable to add an Override annotation.
This method overrides
ReactiveMutationQuery.setParameterList
; it is advisable to add an Override annotation.
*
* @apiNote This is used for binding a list of values to an expression
* such as {@code entity.field in (:values)}.
*
* @return {@code this}, for method chaining
*/
<P> ReactiveQuery<R> setParameterList(int parameter, P[] arguments, BindableType<P> type);
<P> ReactiveQuery<R> setParameterList(int parameter, P[] arguments, Type<P> type);

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
ReactiveSelectionQuery.setParameterList
; it is advisable to add an Override annotation.
This method overrides
ReactiveMutationQuery.setParameterList
; it is advisable to add an Override annotation.
@@ -314,7 +314,7 @@
*
* @return {@code this}, for method chaining
*/
<P> ReactiveQuery<R> setParameterList(QueryParameter<P> parameter, Collection<? extends P> arguments, BindableType<P> type);
<P> ReactiveQuery<R> setParameterList(QueryParameter<P> parameter, Collection<? extends P> arguments, Type<P> type);

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
ReactiveSelectionQuery.setParameterList
; it is advisable to add an Override annotation.
This method overrides
ReactiveMutationQuery.setParameterList
; it is advisable to add an Override annotation.
@@ -359,7 +359,7 @@
*
* @return {@code this}, for method chaining
*/
<P> ReactiveQuery<R> setParameterList(QueryParameter<P> parameter, P[] arguments, BindableType<P> type);
<P> ReactiveQuery<R> setParameterList(QueryParameter<P> parameter, P[] arguments, Type<P> type);

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
ReactiveSelectionQuery.setParameterList
; it is advisable to add an Override annotation.
This method overrides
ReactiveMutationQuery.setParameterList
; it is advisable to add an Override annotation.
@gavinking
Copy link
Member Author

Could you also add the issue to the commit message, please?

The effect of the commit is perfectly well documented in the commit message.

Indeed, there is more information in the commit message than there is in the issue report.

How about we focus on concrete ways to make better software than on this sort of process-obsessed checkbox ticking?

@gavinking
Copy link
Member Author

If I'm worrying about issue numbers in commit messages, my attention is on the wrong things, and I'm not focussed on what actually matters. Bureaucracy does harm.

@DavideD
Copy link
Member

DavideD commented May 14, 2025

So melodramatic... but Ok, I'm not going to waist any more time asking this to you.

@gavinking
Copy link
Member Author

So melodramatic

Hahah guilty :-)

@gavinking gavinking merged commit 3ec4ff9 into hibernate:main May 14, 2025
18 checks passed
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.

Upgrade Hibernate ORM to 7.0.0.CR2
2 participants