Skip to content

fix Solve the compatibility problem between fastjson1 and fastjson2 #3654 - #3661

Merged
wenshao merged 6 commits into
alibaba:mainfrom
mengnankkkk:fix-3654
Jul 27, 2025
Merged

fix Solve the compatibility problem between fastjson1 and fastjson2 #3654#3661
wenshao merged 6 commits into
alibaba:mainfrom
mengnankkkk:fix-3654

Conversation

@mengnankkkk

@mengnankkkk mengnankkkk commented Jul 23, 2025

Copy link
Copy Markdown
Contributor

…3654

What this PR does / why we need it?

Solve the compatibility problem between fastjson1 and fastjson2

Summary of your change

Please indicate you've done the following:

  • Made sure tests are passing and test coverage is added if needed.
  • Made sure commit message follow the rule of Conventional Commits specification.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

@mengnankkkk

mengnankkkk commented Jul 23, 2025

Copy link
Copy Markdown
Contributor Author

单元测试,checkstyle一直改不过来。放弃了。换行符一直卡bug。就用main函数来测试了

@mengnankkkk mengnankkkk changed the title fix Solve the compatibility problem between fastjson1 and fastjson2 #… fix Solve the compatibility problem between fastjson1 and fastjson2 #3654 Jul 23, 2025
@mengnankkkk mengnankkkk reopened this Jul 24, 2025
@wenshao

wenshao commented Jul 24, 2025

Copy link
Copy Markdown
Member

不能用main函数做单测的哈,这样没法保证后续每次集成和发布功能都正确

@mengnankkkk

mengnankkkk commented Jul 24, 2025

Copy link
Copy Markdown
Contributor Author

不能用main函数做单测的哈,这样没法保证后续每次集成和发布功能都正确

正在修改,马上提交

@wenshao

wenshao commented Jul 27, 2025

Copy link
Copy Markdown
Member

处理下代码冲突

@mengnankkkk

Copy link
Copy Markdown
Contributor Author

处理下代码冲突

好的,解决了

@wenshao
wenshao merged commit 569dd50 into alibaba:main Jul 27, 2025
13 checks passed
@mengnankkkk
mengnankkkk deleted the fix-3654 branch July 27, 2025 12:30
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