Skip to content

Add a BSD 3-Clause license - #53

Merged
roytsmart merged 1 commit into
mainfrom
feature/license-bsd-3-clause
Aug 19, 2026
Merged

Add a BSD 3-Clause license#53
roytsmart merged 1 commit into
mainfrom
feature/license-bsd-3-clause

Conversation

@roytsmart

Copy link
Copy Markdown
Contributor

This repository has no license, which means the default of all rights reserved
applies: nobody has permission to use, modify, or redistribute it, even though
it is published on PyPI and imported by esis. This is the same change merged
across the rest of the organization: sun-data/regridding#54,
sun-data/named-arrays#203, sun-data/colorsynth#11, sun-data/ndfilters#28,
sun-data/optika#199, sun-data/ctis#24, sun-data/aastex#17,
sun-data/solar-dynamics-observatory#22, and
sun-data/interface-region-imaging-spectrograph#48.

BSD 3-Clause matches numpy and astropy, which this package sits on top of.
It also satisfies NASA SPD-41a, which asks that software developed with Science
Mission Directorate funding be released under a permissive license, and its
third clause forbids using the copyright holders' names to endorse derived
products.

Changes

  • LICENSE, the standard BSD 3-Clause text, with the copyright line
    Copyright (c) 2024-2026, Roy T. Smart and Jacob D. Parker. Those are the two
    contributors to this repository, and 2024 is when it was created.
  • pyproject.toml declares license = "BSD-3-Clause" and
    license-files = ["LICENSE"], the PEP 639 form.

Verified by building a wheel from this branch: the metadata is
Metadata-Version: 2.4 with License-Expression: BSD-3-Clause and
License-File: LICENSE, so PyPI will display it on the next release. There is
no License :: classifier to conflict with the expression.

Heads up on the Ruff check

Running ruff check . locally with ruff 0.16.3 reports one pre-existing error
unrelated to this change, D100 undocumented-public-module in docs/conf.py.
This repository sets preview = true and selects the D and DOC rule sets,
so the effective rules move with the ruff release, and the last Ruff workflow
run here was in June. If the check fails on this pull request, that is why, and
it needs its own fix rather than being folded in here.

Worth knowing before merging

The copyright line names the authors rather than Montana State University.
MSU's faculty handbook gives authors ownership of copyrightable works other
than commissioned courseware, which is the basis for that, but the university
has not been asked, and if it ever asserts ownership the line is an ordinary
commit to correct. Releasing permissively is the part that does not undo:
copies distributed under these terms stay licensed under them.

🤖 Generated with Claude Code

The repository had no license, so the default of all rights reserved applied:
nobody had permission to use, modify, or redistribute it, even though it is
published on PyPI and imported by `esis`.

Use the BSD 3-Clause license, matching the rest of the organization. It also
satisfies NASA SPD-41a, which asks that software developed with Science Mission
Directorate funding be released under a permissive license.

Declare it in the package metadata using the PEP 639 fields, so that the built
distributions carry `License-Expression: BSD-3-Clause` and ship the `LICENSE`
file.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (d2ae951) to head (982b32f).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #53   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           19        19           
  Lines          840       840           
=========================================
  Hits           840       840           
Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@roytsmart
roytsmart merged commit 864910c into main Aug 19, 2026
11 checks passed
@roytsmart
roytsmart deleted the feature/license-bsd-3-clause branch August 19, 2026 16:07
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