Skip to content

[WIP] Fix action error in PR 1551#1557

Closed
Copilot wants to merge 2 commits intomasterfrom
copilot/fix-pr-1551-action-error
Closed

[WIP] Fix action error in PR 1551#1557
Copilot wants to merge 2 commits intomasterfrom
copilot/fix-pr-1551-action-error

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 26, 2026

  • Investigate CI failure in PR [AI-8th] feat: add Apache Fory serialization support for SOFARPC #1551 (formatter modifies ForySerializerTest.java causing check_format.sh to fail)
  • Get all files from the PR branch (codec-sofa-fory module)
  • Create codec/codec-sofa-fory module with all source files:
    • ForySerializer.java (main serializer, registered as extension "fory" with code 23)
    • SofaRequestForySerializer.java (custom serializer for SofaRequest)
    • SofaResponseForySerializer.java (custom serializer for SofaResponse)
    • SPI registration file
  • Create test files following project's formatting conventions (matching FurySerializerTest.java style)
  • Create test model classes (DemoRequest, DemoResponse, DemoService, BlackListClass, etc.)
  • Generate binary test data files for xlang interoperability tests
  • Update bom/pom.xml with fory.version=0.16.0 and Apache Fory dependency
  • Update codec/pom.xml to include codec-sofa-fory module
  • Run Maven formatter to verify all files are pre-formatted correctly (0 files needed reformatting)
  • Verified git status --untracked-files=no --porcelain shows no uncommitted changes (check_format.sh would pass)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI requested a review from sunhailin-Leo March 26, 2026 11:50
Copilot stopped work on behalf of sunhailin-Leo due to an error March 26, 2026 11:50
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