Skip to content

Replace legacy ALPS license headers with MIT SPDX notices - #109

Draft
skilledwolf wants to merge 1 commit into
masterfrom
cleanup/mit-license-headers
Draft

Replace legacy ALPS license headers with MIT SPDX notices#109
skilledwolf wants to merge 1 commit into
masterfrom
cleanup/mit-license-headers

Conversation

@skilledwolf

Copy link
Copy Markdown
Collaborator

Summary

This PR aligns files that still refer to the historical ALPS Library or Application licenses with the repository-level MIT license.

Related discussion: #108

Changes

  • Replace legacy ALPS license boilerplate with concise SPDX-License-Identifier: MIT notices.
  • Remove obsolete preamble templates and scripts that generated the old headers.
  • Remove the legacy license check from alps_inspect.
  • Update user-visible license messages and documentation to reference MIT.
  • Make the lattice-preview About dialog load the canonical LICENSE.txt rather than embedding another copy.

Why

The repository-level LICENSE.txt declares MIT, while a number of older source files, tools, and tutorials still named the superseded ALPS licenses and, in some cases, described different obligations. This PR removes that contradiction while keeping the canonical license text in one place.

Scope

Third-party files that already contain their own complete license notices are intentionally unchanged. Those notices may need to remain for attribution and license compliance.

Validation

  • Confirmed that the targeted legacy ALPS license phrases no longer remain.
  • Validated Python syntax and notebook JSON.
  • Ran whitespace and staged-diff checks.

Follow-up

Maintainer confirmation in #108 is requested to verify that the affected per-file notices are historical rather than intentional licensing exceptions.

@skilledwolf

Copy link
Copy Markdown
Collaborator Author

Note: I cancelled the CI runs, since this PR doesn't functionally change anything (just comments).

@skilledwolf
skilledwolf force-pushed the cleanup/mit-license-headers branch 3 times, most recently from 5af82a7 to 5ab830f Compare July 23, 2026 13:08
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.

1 participant