Skip to content

fixed org.pkl.spring.boot.PklAutoConfiguration for Spring Boot 3.x#16

Merged
bioball merged 2 commits into
apple:mainfrom
protobufel2:autoconfig-fix2
Oct 8, 2025
Merged

fixed org.pkl.spring.boot.PklAutoConfiguration for Spring Boot 3.x#16
bioball merged 2 commits into
apple:mainfrom
protobufel2:autoconfig-fix2

Conversation

@protobufel2
Copy link
Copy Markdown
Contributor

see #14

Comment thread src/main/java/org/pkl/spring/boot/PklAutoConfiguration.java
Comment thread src/main/resources/META-INF/spring.factories Outdated
Copy link
Copy Markdown
Contributor Author

@protobufel2 protobufel2 left a comment

Choose a reason for hiding this comment

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

  • updated Spring/Spring Boot to the latest version
  • resolved minor requests (added new lines)
  • applied spotlessApply
  • commented regarding Spring Boot 2 compatibility - I think it's compatible.

Comment thread src/main/java/org/pkl/spring/boot/PklAutoConfiguration.java
@protobufel2
Copy link
Copy Markdown
Contributor Author

@bioball Please approve so people can use it with Spring Boot 3 for sure :)

Copy link
Copy Markdown
Member

@bioball bioball left a comment

Choose a reason for hiding this comment

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

LGTM

@bioball bioball merged commit da60cb0 into apple:main Oct 8, 2025
0 of 2 checks passed
Forbiddem pushed a commit to Forbiddem/pkl-spring that referenced this pull request May 17, 2026
The CircleCI badge in README.adoc has been broken since CI moved to
GitHub Actions (apple#19) and the CircleCI config was removed (apple#20). Other
apple/pkl-* repos do not carry a CI status badge in their README, so
drop it rather than substitute a new one.

PklAutoConfiguration is already declared in
META-INF/spring/org.springframework.boot.autoconfigure.AutoConfiguration.imports
since apple#16, which makes the matching EnableAutoConfiguration entry in
spring.factories redundant. The PropertySourceLoader entry is preserved
because Spring Boot still loads PropertySourceLoader implementations
exclusively from spring.factories.
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