Skip to content

Fixes to generation of AUTHORS file

Karl Wette requested to merge ANU-CGA/lalsuite:fix-AUTHOR-generation into master

Description

  • Exclude merge commits from author list
    • So that e.g. if someone merges in master branch locally, which brings in changes from others, those don't count towards authorship.
    • This removes a few names from union of all AUTHORS files. Confirmed that these authors only performed merge commits, or else contributed to code no longer part of LALSuite (e.g. gracedb/, laldetchar/).
  • Optionally find authors using git blame
    • Ran this once and committed the results
    • Should only be necessary after major restructuring
  • Rename lal*/.AUTHORS to lal*/.historical_authors
    • To make the purpose of those files clearer

Closes #665 (closed)

Required by !2128 (merged)

Requires !2130 (merged)

API Changes and Justification

Backwards Compatible Changes

  • This change does not modify any class/function/struct/type definitions in a public C header file or any Python class/function definitions
  • This change adds new classes/functions/structs/types to a public C header file or Python module

Backwards Incompatible Changes

  • This change modifies an existing class/function/struct/type definition in a public C header file or Python module
  • This change removes an existing class/function/struct/type from a public C header file or Python module

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

Please provide details on any reviews related to this change and and the associated reviewers.

Edited by Karl Wette

Merge request reports