Skip to content

Conversation

@pjfanning
Copy link
Member

Purpose of the pull request

Upgrade to latest available version of POI

What's changed?

Checklist

  • I have read the Contributor Guide.
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

@github-actions
Copy link

github-actions bot commented Dec 5, 2025

Test Results

249 tests   249 ✅  2m 15s ⏱️
 50 suites    0 💤
 50 files      0 ❌

Results for commit f35fba3.

♻️ This comment has been updated with latest results.

@psxjoy psxjoy self-requested a review December 5, 2025 09:31
psxjoy
psxjoy previously approved these changes Dec 5, 2025
Copy link
Member

@psxjoy psxjoy left a comment

Choose a reason for hiding this comment

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

lgtm

@psxjoy
Copy link
Member

psxjoy commented Dec 5, 2025

hi @pjfanning , we use spotless to ensure consistent code formatting. You may need to run mvn spotless:apply when submitting a PR.

我们使用spotless 来确保代码格式的统一。或许需要您在提交 PR 的时候,执行一下mvn spotless:apply

@pjfanning
Copy link
Member Author

@psxjoy I have now fixed the spotless format issue.

Copy link
Member

@delei delei left a comment

Choose a reason for hiding this comment

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

LGTM

@psxjoy psxjoy merged commit 945b543 into main Dec 5, 2025
11 checks passed
@psxjoy psxjoy deleted the pjfanning-patch-1 branch December 5, 2025 09:49
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