Standardize copyright headers and licensing#4
Merged
Conversation
- Replace outdated GPL-3 AdvantageKit-License.md with current BSD 3-Clause text (AKit relicensed from GPL-3 to BSD via contributor consent in May 2025) - Update 11 source files with stale FRC 6328/GPL-3 headers to BSD (Littleton Robotics) - Add BSD 3-Clause copyright headers to 62 Triple Helix-original source files - Add LICENSE file (BSD 3-Clause, Triple Helix + Littleton Robotics) - Add Credits and Licensing section to README Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Files with no meaningful AKit lineage get TH-only copyright (Constants, DriveConstants, GyroIOBoron, VisionFilter, VisionConstants). Files with substantial TH rewrites over an AKit skeleton get joint copyright (Drive, DriveCommands, Vision, Module, VisionIO, VisionIOPhotonVision, VisionIOPhotonVisionSim). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- LocalADStarAK.java: restore Littleton-only header; code body is identical to the AKit original (only package path changed) - Robot.java: upgrade to joint header; AKit skeleton (BuildConstants logging, mode switch, Logger.start(), CommandScheduler boilerplate) forms the structural foundation - Constants.java: upgrade to joint header; the Mode enum and simMode/currentMode pattern are verbatim from AKit; TH added all hardware constants around it Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The root LICENSE file now covers both Littleton Robotics and Triple Helix copyright. Update the README link accordingly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Audits every file with a Littleton Robotics copyright header against the actual AdvantageKit template originals and assigns accurate copyright statements based on true authorship.
Three-tier header scheme
Main.java,PhoenixUtil.java,SparkUtil.java,LocalADStarAK.java,GyroIO.java,ModuleIO.java,ModuleIOSimWPI.java,ModuleIOTalonFX.java,PhoenixOdometryThread.java— identical or trivially adapted from AKitRobot.java,Constants.java,Drive.java,DriveCommands.java,Vision.java,Module.java,VisionIO.java,VisionIOPhotonVision.java,VisionIOPhotonVisionSim.java— AKit skeleton with substantial TH additionsDriveConstants.java,GyroIOBoron.java,VisionFilter.java,VisionConstants.java, and all other TH-originated files — no meaningful AKit lineageAlso replaced outdated GPL-3 headers with BSD-3-Clause, created
LICENSE, updatedAdvantageKit-License.md, and added a Credits section toREADME.md.Test plan
./gradlew buildpasses🤖 Generated with Claude Code