Skip to content

Conversation

@alaahong
Copy link
Member

Purpose of the pull request

Closed: #603

What's changed?

refactor: rename resources and classes for improved organization and clarity

Checklist

  • I have read the Contributor Guide.
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

@alaahong
Copy link
Member Author

@psxjoy @delei please feel free to discuss the module name and parent reference.

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 refactors the Apache Fesod project by renaming modules, artifacts, and resources from "fastexcel" to "fesod" for improved organization and clarity as part of the Apache incubation process.

  • Updates Maven group ID from cn.idev.excel to org.apache.fesod
  • Renames artifacts from fastexcel-* to fesod-*
  • Removes the fastexcel-support module entirely
  • Updates CI/CD workflows to reference the renamed modules

Reviewed Changes

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

Show a summary per file
File Description
fesod/pom.xml Updates parent group ID and artifact names to use fesod naming
fesod-examples/pom.xml Updates parent and dependency references to use new fesod naming
fastexcel-support/pom.xml Removes the entire fastexcel-support module
.github/workflows/fuzz-tests.yml Updates module references in test commands from fastexcel to fesod
.github/workflows/ci.yml Updates Maven build and test commands to reference fesod module

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@delei delei left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@psxjoy psxjoy left a comment

Choose a reason for hiding this comment

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

LGTM

@psxjoy psxjoy merged commit 07fbf0a into apache:main Sep 27, 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.

[Task] move code package into apache

3 participants