Skip to content

Conversation

olafmat
Copy link
Contributor

@olafmat olafmat commented Mar 10, 2025

What this PR does / why we need it?

It's just fixing of a method name which contained a type. The backward compatibility is preserved.

Summary of your change

This PR changes method name forEchArrayObject to forEachArrayObject because it contained a typo.
The previous method is left for compatibility, but marked as deprecated.

The method was not used in the tests or documentation.

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.

@CLAassistant
Copy link

CLAassistant commented Mar 10, 2025

CLA assistant check
All committers have signed the CLA.

@wenshao
Copy link
Member

wenshao commented Mar 11, 2025

fastjson2需要兼容JDK8,你提交的代码无法在JDK8下构建通过

@olafmat
Copy link
Contributor Author

olafmat commented Mar 11, 2025

Sorry. It's already fixed, the code is compatible with JDK 8 now.

@wenshao wenshao merged commit d44e4a4 into alibaba:main Mar 12, 2025
13 checks passed
@wenshao wenshao added this to the 2.0.57 milestone Mar 12, 2025
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