Repository navigation
chore: one CONTRIBUTING and one LICENSE for the repository - #1396
Merged
Merged
Conversation
elibol
requested review from
binarybana,
cryos,
kkraus14,
nihalpasham,
roivanov and
rparolin
as code owners
October 7, 2026 22:26
cutile-rs/CONTRIBUTING.md repeated the DCO text and sign-off instructions from the root guide and said nothing about the monorepo. Fold what was specific to it into the root CONTRIBUTING.md: the branch and PR naming convention (with docs rather than doc, which is what main's history uses) and how to run the Tile test scripts from cutile-rs/. AGENTS.md and the cutile-rs README point at the root guide. The LICENSE copies under cutile-rs/ and cuda-oxide/ duplicated the root file and nothing consumed them: every crate declares its license through the SPDX license field rather than license-file, GitHub reads the root LICENSE, and Apache-2.0 asks for one copy per distribution. The README's per-component license table collapses to a sentence, and cuda-oxide's README and THIRD_PARTY_NOTICES point at the root file. Signed-off-by: Melih Elibol <1884921+elibol@users.noreply.github.com>
elibol
force-pushed
the
chore/one-contributing-one-license
branch
from
October 7, 2026 22:28
e93b8bc to
c6a1c71
Compare
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
2 participants
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.
Both component trees carried their own CONTRIBUTING.md and LICENSE next to the root ones. This leaves one of each.
cutile-rs/CONTRIBUTING.mdrepeated the DCO text and sign-off instructions from the root guide and said nothing about the monorepo. The two things it had that the root guide lacked move there: the branch and PR naming convention (withdocsrather thandoc, which is what the history onmainuses) and how to run the Tile test scripts fromcutile-rs/. The root guide's Toolchains section, AGENTS.md, and the cutile-rs README now point at the root guide.The LICENSE copies under
cutile-rs/andcuda-oxide/duplicated the root file, and nothing consumed them: every crate declares its license through the SPDXlicensefield rather thanlicense-file, GitHub reads the root LICENSE, and Apache-2.0 asks for one copy per distribution. The root README's per-component license table collapses to one sentence, and the two cuda-oxide references (its README and THIRD_PARTY_NOTICES) point at the root file. The cuda-oxide copy was added deliberately in the recent scoping commit (bb8d4fa), so this is the place to say so if a local copy is wanted after all.No code changes.
check-source-references.shandcheck-spdx-headers.shpass.