[Task] Correct source attribution and align copyright years following LEGAL-737

オープン
#1,322 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@himanchalkaushale がすでに取り組んでいます。

2026年11月10日 から。

評価

難易度
5/5
見積もり時間
1週間以上
初心者へのやさしさ
35/100
issue の種類
リファクタリング
明瞭さ
おおむね明確
活発さ
活発
技術スタック
java

調査の方向性

Start by checking the provenance of the 12 named classes in fesod-common and fesod-sheet, then inspect other EasyExcel files that already carry ASF headers. Use upstream references to decide which attributions apply and reconcile source header years with NOTICE. Done means the attribution and year ranges are consistent, license headers are preserved, and provenance evidence and remediation links are recorded; the issue also asks for a report to LEGAL-737.

索引モデルが issue の本文から書いたものです。

説明

good first issue
Motivation

This issue tracks the implementation follow-up to #1106 based on Justin Mclean's review in LEGAL-737.

Justin agreed that retaining Alibaba attribution in EasyExcel-derived files is acceptable (option A / the Pekko approach), because the EasyExcel code is outside the SGA scope. The current layout—an ASF header followed by Alibaba attribution in a separate comment block—is acceptable, and LICENSE needs no changes.

However, his review identified incorrect attribution on ASF-origin files and inconsistent copyright years. He also clarified that the Alibaba lines were added by the Fesod project rather than inherited from upstream EasyExcel. Their placement therefore needs to be checked against actual code provenance.

How to complete
  1. Audit files carrying Alibaba attribution and distinguish:
    • EasyExcel-derived code, where Alibaba attribution may remain.
    • New FastExcel/Fesod code covered by the SGA, where Alibaba attribution should not appear.
    • Code originating from Apache POI or Apache Commons, where the incorrectly added Alibaba line should be removed and the ASF header retained.
  2. Start with the files Justin identified below, then check other EasyExcel files that already carried an ASF header. Record upstream references supporting the changes.
  3. Reconcile the source-header year range (2018–2024) with NOTICE (2022–2024). Choose a consistent range supported by the provenance review; the comment does not prescribe which range to use.
  4. Keep the accepted header layout for applicable EasyExcel-derived files. Restoring @author tags is not required by the review.

Files identified by Justin (not an exhaustive list):

Module Classes
fesod-common StringUtils, BooleanUtils, ListUtils, MapUtils, MemberUtils
fesod-sheet Validate, DateUtils, DataFormatter, FileUtils, ClassUtils, FieldUtils, SharedStringsTableHandler
Task List
  • Verify the provenance of the 12 identified classes and remove their incorrect Alibaba attribution.
  • Check for additional ASF-origin files with the same attribution problem.
  • Ensure Alibaba attribution is limited to applicable EasyExcel-derived code and absent from new SGA-covered FastExcel/Fesod code.
  • Make the relevant source-header and NOTICE year ranges consistent, documenting the basis for the selected range.
  • Verify the diff preserves ASF license headers and makes no functional changes; pass the repository's formatting and license-header checks.
  • Link the remediation PR(s) and provenance evidence here and in #1106.
  • Report the completed changes back to LEGAL-737 and request confirmation that the remaining concerns are addressed.
Anything else?
  • Related decision tracking: #1106
  • Legal discussion and review: LEGAL-737

This issue tracks the concrete attribution cleanup requested in the review. It does not assume that LEGAL-737 has been formally resolved or that graduation approval has been granted.

主要言語
Java
スター
6.3k
フォーク
554
平均マージ
1日 21時間
マージ済み PR(30日)
44

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

apache/fesod のほかの issue

apache/fesod の issue をすべて見る

似ている issue

Java の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。