Skip to content

Conversation

@GOODBOY008
Copy link
Member

Description

This PR renames classes to align with the project's rebranding from FastExcel to Fesod.

Changes

  • Rename FastExcelConstants to FesodConstants
  • Rename FastExcelTempFileCreationStrategy to FesodTempFileCreationStrategy
  • Update all references across the codebase (imports, usages, and instantiations)

Files Modified

  • fesod-sheet/src/main/java/org/apache/fesod/sheet/constant/FesodConstants.java (renamed)
  • fesod-sheet/src/main/java/org/apache/fesod/sheet/util/FesodTempFileCreationStrategy.java (renamed)
  • fesod-sheet/src/main/java/org/apache/fesod/sheet/util/FileUtils.java
  • fesod-sheet/src/main/java/org/apache/fesod/sheet/metadata/data/ReadCellData.java
  • fesod-sheet/src/main/java/org/apache/fesod/sheet/analysis/v07/handlers/CellTagHandler.java
  • fesod-sheet/src/main/java/org/apache/fesod/sheet/analysis/v03/handlers/FormulaRecordHandler.java

- Rename FastExcelConstants to FesodConstants
- Rename FastExcelTempFileCreationStrategy to FesodTempFileCreationStrategy
- Update all references across the codebase
Copilot AI review requested due to automatic review settings December 30, 2025 09:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR performs a systematic refactoring to rename classes from "FastExcel" to "Fesod" as part of the project's rebranding effort. The changes are consistent and comprehensive across the codebase.

Key Changes:

  • Renamed FastExcelConstants class to FesodConstants
  • Renamed FastExcelTempFileCreationStrategy class to FesodTempFileCreationStrategy including its constructors
  • Updated all imports, usages, and code comments to reflect the new class names

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
FesodConstants.java Renamed class from FastExcelConstants to FesodConstants
FesodTempFileCreationStrategy.java Renamed class and constructors from FastExcelTempFileCreationStrategy to FesodTempFileCreationStrategy, updated inline comment reference
FileUtils.java Updated instantiation to use FesodTempFileCreationStrategy
ReadCellData.java Updated import and usage of FesodConstants
CellTagHandler.java Updated import and usage of FesodConstants
FormulaRecordHandler.java Updated import and usage of FesodConstants

The refactoring has been executed thoroughly with all class names, constructor names, imports, usages, and comments properly updated to maintain consistency throughout the codebase.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GOODBOY008
Copy link
Member Author

@alaahong @delei PTAL,Thank you ~

Copy link
Member

@alaahong alaahong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename might good, but seems some changes are not the common value under Fesod, seems limited to excel part.

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