Skip to content

Releasing v2.2.1 #43

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.

Sign up for GitHub

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

Jump to bottom
Merged
makbn merged 44 commits into from
Aug 28, 2025
Merged

Releasing v2.2.1 #43

makbn merged 44 commits into from
Aug 28, 2025

Conversation

@makbn
Copy link
Owner

@makbn makbn commented Aug 27, 2025 •
edited
Loading

Release: JThumbnailer v2.2.1

This PR merges 43 commits from dev/2.2.1 into release/2.2.1 to prepare for the v2.2.1 release.
It includes dependency upgrades, configuration refactoring, style enforcement, and critical bug fixes.

A special thanks to @FredPraca for his extensive contributions and efforts in this release.

What's New

  • Dependencies & Build

    • Upgraded dependencies and fixed compilation errors — (@FredPraca)
    • Updated OpenOffice version — (@makbn)
    • Updated github-action-build.yml for CI/CD improvements — (@makbn, @FredPraca)
    • Improved test configuration setup — (@makbn)

  • Code Quality & Style

    • Added Spotless Gradle plugin to enforce consistent code style — (@FredPraca)
    • Standardized formatting, import ordering, and applied IntelliJ formatter — (@FredPraca, @makbn)
    • Migrated example configuration to YAML format — (@FredPraca, @makbn)

  • Refactoring & Configuration

    • Migrated configuration properties into structured classes — (@FredPraca)
    • Improved property naming (port → ports) for clarity — (@FredPraca, @makbn)
    • Applied stricter typing for directories and configurations — (@FredPraca)
    • Updated README to reflect minimum Java version requirements and property changes — (@FredPraca, @makbn)
    • Renamed exceptions for consistency (ThumbnailerException → ThumbnailException, etc.) — (@makbn)

  • Bug Fixes

    • Fixed NullPointerException when working directory is not provided (now defaults to OS temp dir) — (@FredPraca)
    • Fixed conflicts between Apache Commons Logging and Jakarta Commons Logging — (@makbn)
    • Fixed Spring Modulith ApplicationModuleListener event handling issue — (@makbn)
    • Prevented accidental deletion of system temp directories — (@makbn)
    • Removed ExcelConverterThumbnailer.java and downgraded iceblue to 5.3.1 for stability — (@makbn)
    • Fixed javadoc generation and packaged docs with sources — (@makbn, @FredPraca)

  • Other Improvements

    • Introduced pipe-based configuration for OpenOffice instances — (@makbn)
    • Added more detailed error traces for debugging — (@makbn)
    • Updated environment configurations for tests and builds — (@makbn)
    • Limited test context for OpenOffice for stability — (@makbn)
    • Improved thread safety in internal maps with size limits — (@makbn)

Note

This PR description was AI-generated and may be updated if any discrepancies are found regarding tasks, contributor names, or issue details.

FredPraca and others added 30 commits August 20, 2025 17:26
@FredPraca
feat(upgrade): upgrade dependencies versions and fix compilation errors
88ce5e7
@FredPraca
fix(warnnings): remove useless annotations
467a044
@makbn @FredPraca
updated open office version
515abab
@makbn @FredPraca
updated github-action-build.yml
39b60ab
@makbn @FredPraca
Update github-action-build.yml
7947883
@makbn @FredPraca
Update README.md
7e043b6
@FredPraca
fix(readme): update README according to version change and minimum Ja…
59effcb 
…va requirement
@makbn
Merge pull request #24 from FredPraca/feat/upgrade-versions
365811b 
Upgrade versions
@FredPraca
fix(style): add Spotless Gradle plugin and fix style
b69c4a6
@FredPraca
fix(style): change spotless section to be more readable
8144781
@makbn
Merge pull request #28 from FredPraca/feat/standardize-style
9138039 
[Style] Standardize and enforce code style
@makbn
fixed the issue with multiple spring-modulith-bom imports.
d22f935
@makbn
fixed the issue with missing imports
a08234f
@makbn
fixed the issue with spring module event. in the newer version Applic…
632a20b 
…ationModuleListener enforces transaction while for this care we don't have and need one. defaultExecution for EventListener is true.
@makbn
fixed the conflict between apache commons logging and jakarta commons…
9035884 
… logging
@FredPraca
Merge pull request #1 from makbn/dev/2.2.1
539c717 
Retrieved changes from upstream
@FredPraca
feat(refactor): prepare properties and configuration
5f96caf 
fix(style): use spaces instead of tabs
@FredPraca
feat(refactoring): use configuration properties and split configurations
b750f75 
update code and properties files
@FredPraca
feat(refactoring): provide YAML configuration file for example
6c538a0 
this file is the YAML version of the properties file currently used
@FredPraca
feat(semantic): rename to match the properties content
1ebfb4d
@FredPraca
feat(refactoring): use more specific type for directories
06c455e 
add temporary component to working dirs
deal with files instead of strings
@FredPraca
fix(style): apply Spotless
20711a0
@makbn
Merge pull request #33 from FredPraca/feat/refactor-application-prope…
9e59f9e 
…rties

Feat/refactor application properties
@makbn
renamed ThumbnailerException.java to ThumbnailException. renamed Thum…
9ac55eb 
…bnailerRuntimeException.java to ThumbnailRuntimeException.java. used FieldDefault annotation. added max size to the waiting map and made it thread safe.
@makbn
fixed minor api usage issues.
82e230b
@FredPraca
feat(standardize): update property to match purpose
704660b 
transform from port to ports as this is a list of ports.
@FredPraca
fix(doc): update to match the change in properties
5c9275f
@FredPraca
fix(style): change configuration to stick to IntelliJ formatter
d91010f 
change the way import handling is done as Palantir order is diffrent from the one we want
add order to group imports
@FredPraca
fix(docs): fix javadoc generation and package it along sources
9c5d859
@makbn
Merge pull request #34 from FredPraca/feat/standardize-property-name
b6a102a 
Feat/standardize property name
makbn and others added 13 commits August 25, 2025 08:29
@makbn
Merge pull request #36 from FredPraca/fix/fix-and-publish-javadocs-an…
d986f8a 
…d-sources

fix(docs): fix javadoc generation and package it along sources
@FredPraca
fix(npe): use the OS temp directory when working dir is not provided
ce4f6fd 
fix resource for unit tests
@FredPraca
fix(style): as previous merge relied on a different spotless configur…
b40e0c6 
…ation, style must be fixed
@makbn
Merge pull request #39 from FredPraca/dev/2.2.1
0b961e9 
fix(style): as previous merge relied on a different spotless configur…
@makbn
Merge pull request #38 from FredPraca/fix/npe_missing_working_dir
9b8b328 
fix(npe): use the OS temp directory when working dir is not provided
@makbn
changed test config file to yml. added pipe names to be able to run o…
4518f14 
…ffice instance using pipes. added a check to prevent any unnecessary try to temove system temp directory. updated
@makbn
added pipe to test class. updated github action for build.
ca58a3f
@makbn
added trace to print out error messages
baf35b9
@makbn
updated envs
4097c02
@makbn
updated envs with lib path
29cb995
@makbn
limited context for OpenOfficeTest
b4d2bab
@makbn
Removed ExcelConverterThumbnailer.java. downgraded iceblue to 5.3.1. …
16edbfc 
…added context class loader to the worker threads.
@makbn
Merge pull request #41 from makbn/dev/2.2.1_add_pipe
e5af68e 
Dev/2.2.1 add pipe
@makbn makbn self-assigned this Aug 27, 2025
@makbn makbn added this to the 2.2.1 milestone Aug 28, 2025
@makbn makbn merged commit 043e568 into release/2.2.1 Aug 28, 2025
1 of 4 checks passed
@makbn makbn added the Release label Aug 28, 2025
@makbn makbn mentioned this pull request Aug 28, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@makbn makbn

Labels

Release

Projects

None yet

Milestone

2.2.1

Development

Successfully merging this pull request may close these issues.

3 participants

@makbn @FredPraca