Skip to content

[NETBEANS-3558] Spring Framework 5.2.2#1756

Merged
matthiasblaesing merged 1 commit intoapache:masterfrom
hectorespert:spring_framework_5
Dec 25, 2019
Merged

[NETBEANS-3558] Spring Framework 5.2.2#1756
matthiasblaesing merged 1 commit intoapache:masterfrom
hectorespert:spring_framework_5

Conversation

@hectorespert
Copy link
Member

@hectorespert hectorespert commented Dec 7, 2019

Update java/libs.springframework module to add Spring Framework 5.2.2 library dependency for ant projects. NETBEANS-3558

Screenshot from 2019-12-22 13-24-24

@hectorespert hectorespert force-pushed the spring_framework_5 branch 4 times, most recently from 78a05e9 to ee90f35 Compare December 8, 2019 14:31
@hectorespert hectorespert marked this pull request as ready for review December 8, 2019 14:34
@hectorespert hectorespert force-pushed the spring_framework_5 branch 2 times, most recently from e0f49af to 69f7340 Compare December 24, 2019 11:24
Copy link
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

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

Thank you - looks sane, for the license + notice they can be handled more compact. See inline comment.

Version: 5.2.2.RELEASE
Origin: Spring
License: Apache-2.0-spring5.2.2
URL: https://spring.io/projects/spring-framework
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a group of dependencies with the same license + notice (my assumption). If so, please use the "Files" header to reference all files the license+notice apply to and remove the additional license + notice files. That way it is better readable.

Copy link
Member Author

Choose a reason for hiding this comment

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

Changed

Copy link
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

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

Looks sane. Thank you.

@matthiasblaesing matthiasblaesing merged commit 929890e into apache:master Dec 25, 2019
@junichi11 junichi11 added this to the 11.3 milestone Dec 25, 2019
@hectorespert hectorespert deleted the spring_framework_5 branch February 8, 2020 16:20
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.

3 participants