Homec4science

Delete license headers from files

Authored by vrana <jakubv@fb.com> on Nov 5 2012, 05:25.

Description

Delete license headers from files

Summary:
This commit doesn't change license of any file. It just makes the license implicit (inherited from LICENSE file in the root directory).

We are removing the headers for these reasons:

  • It wastes space in editors, less code is visible in editor upon opening a file.
  • It brings noise to diff of the first change of any file every year.
  • It confuses Git file copy detection when creating small files.
  • We don't have an explicit license header in other files (JS, CSS, images, documentation).
  • Using license header in every file is not obligatory: http://www.apache.org/dev/apply-license.html#new.

This change is approved by Alma Chao (Lead Open Source and IP Counsel at Facebook).

Test Plan: Verified that the license survived only in LICENSE file.

Reviewers: epriestley, davidrecordon

Reviewed By: epriestley

CC: aran, Korvin

Maniphest Tasks: T2035

Differential Revision: https://secure.phabricator.com/D3887

Details

Committed
vrana <jakubv@fb.com>Nov 5 2012, 20:16
Pushed
aubortMar 17 2017, 12:03
Parents
rPHUc6d16500303f: Recognize flex35 in xhpast
Branches
Unknown
Tags
Unknown

Event Timeline

vrana <jakubv@fb.com> committed rPHU4f9915ecac81: Delete license headers from files (authored by vrana <jakubv@fb.com>).Nov 5 2012, 20:16