Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 28, 2025

  • Update dependencies to latest stable versions:
    • Apache MINA Core: 2.0.1 → 2.0.27
    • Protocol Buffers Java: 2.3.0 → 3.25.5
    • SLF4J API and Simple: 1.6.1 → 2.0.16
  • Add Maven compiler plugin configuration for Java 11
  • Add JUnit 5 and Mockito test dependencies
  • Create unit tests for core classes:
    • BoundedInputStreamTest (8 tests)
    • ProtoBufEncoderTest (3 tests)
    • ProtoBufDecoderTest (5 tests)
    • ProtoBufCoderFactoryTest (3 tests)
    • ProtoBufCoderFilterTest (2 tests)
  • Clean up Eclipse IDE files (add to .gitignore)
  • Improve README.md with:
    • Project description and CI status badge
    • Build instructions
    • Usage examples
    • Dependencies table
    • Accurate project status (PoC, not actively maintained)
    • Code formatting instructions
  • Add GitHub Actions CI workflow (tests Java 11, 17, 21)
  • Fix deprecated API usage (IoSession.close → closeNow)
  • Fix typos in comments
  • Add code formatting with Spotless + Google Java Format
    • Applied to all 11 Java files
    • CI enforces formatting check before build
  • Run code review - passed
  • Run security scan - passed (added explicit permissions to workflow)
Original prompt

Clean up, make a better readme with proper status. Make sure it works and runs. Try updating deps etc if relevant


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits November 28, 2025 04:45
Co-authored-by: meros <450310+meros@users.noreply.github.com>
Co-authored-by: meros <450310+meros@users.noreply.github.com>
@meros
Copy link
Owner

meros commented Nov 28, 2025

@ @copilot it's not actively used and maintained, don't say that in readme

Co-authored-by: meros <450310+meros@users.noreply.github.com>
Copilot AI changed the title [WIP] Update and enhance README with proper status and clarity Modernize project: update deps, add tests, CI, and improve docs Nov 28, 2025
Copilot AI requested a review from meros November 28, 2025 04:56
Co-authored-by: meros <450310+meros@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 28, 2025

@ @copilot it's not actively used and maintained, don't say that in readme

Updated the Status section to say "This is a proof-of-concept project. It is not actively maintained but contributions are welcome." (e2a07ce)

@meros
Copy link
Owner

meros commented Nov 28, 2025

@copilot Make sure all code is well formatted, add a standard formatting system and test for this in the ci

Co-authored-by: meros <450310+meros@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 28, 2025

@copilot Make sure all code is well formatted, add a standard formatting system and test for this in the ci

Added Spotless Maven plugin with Google Java Format. All 11 Java files have been formatted, and CI now enforces format checking before builds. Run mvn spotless:apply to auto-format, mvn spotless:check to verify. (f0aedf1)

Copilot AI changed the title Modernize project: update deps, add tests, CI, and improve docs Modernize project: update deps, add tests, CI, formatting, and improve docs Nov 28, 2025
@meros meros marked this pull request as ready for review November 28, 2025 08:47
@meros meros merged commit ee75025 into master Nov 28, 2025
8 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.

2 participants