Skip to content

build: declare SPDX license and pin hatchling - #130

Merged
mc-nv merged 1 commit into
mainfrom
mchornyi/TRI-1775/update-metadata-version
Sep 15, 2026
Merged

mc-nv merged 1 commit into
mainfrom
mchornyi/TRI-1775/update-metadata-version

Conversation

@mc-nv

@mc-nv mc-nv commented Sep 11, 2026 •

Copy link
Copy Markdown
Contributor

What does the PR do?

  • Declares license = "BSD-3-Clause" so METADATA carries a License-Expression field; the package previously shipped with no license metadata at all.
  • Pins the previously unbounded hatchling requirement so emitted metadata does not depend on what the build environment resolves.

license-files is deliberately omitted because this repo has no LICENSE file — which is also why the add-license pre-commit hook is disabled here.

Where should the reviewer start?

pyproject.toml.

Test plan:

Verified: Metadata-Version 2.4 with License-Expression: BSD-3-Clause.

Related PRs:

Related Issues:

  • Resolves: TRI-1775
CI (internal): [#67460724](http://tritonserver.local/ci/pipelines/67460724)

@greptile-apps

greptile-apps Bot commented Sep 11, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 3/5

The PR is not yet safe to merge because request-controlled image downloads can still hold a synchronous preprocessing worker indefinitely.

Summary

  • Adds License-Expression package metadata through license = "BSD-3-Clause".
  • Pins the Hatchling build backend.
  • Aligns CI with the pre-commit Ruff version.
  • Adds image-download status checking, inactivity timeout, and size limiting, but lacks a total fetch deadline.

Reviews (2) · Last reviewed commit: "build: declare SPDX license and pin hatc..."

Declares license = "BSD-3-Clause" so METADATA carries a
License-Expression field; the package previously shipped with no
license metadata at all. license-files is deliberately omitted because
this repo has no LICENSE file, which is also why the add-license
pre-commit hook is disabled here.

Pins the previously unbounded hatchling requirement so the emitted
metadata does not depend on whatever the build environment resolves,
matching the other Triton wheels.

Verified: Metadata-Version 2.4 with License-Expression: BSD-3-Clause.

Refs: TRI-1775
@mc-nv
mc-nv force-pushed the mchornyi/TRI-1775/update-metadata-version branch from 62ea505 to ffe4297 Compare September 14, 2026 18:25
@mc-nv
mc-nv merged commit 29a9f5f into main Sep 15, 2026
4 of 5 checks passed

This branch was successfully deployed

No deployments
GITLAB — ffe4297e Deployed Sep 14, 2026 by mc-nv via mirror_repo #191
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Build system or external dependencies (build: PRs)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants