Skip to content

chore: add license metadata and classifiers to pyproject.toml#8

Merged
Aadil-5122 merged 1 commit intomainfrom
vorflux/license-metadata
Apr 8, 2026
Merged

chore: add license metadata and classifiers to pyproject.toml#8
Aadil-5122 merged 1 commit intomainfrom
vorflux/license-metadata

Conversation

@Aadil-5122
Copy link
Copy Markdown
Contributor

Summary

Add license metadata and Python classifiers to pyproject.toml for consistency with hydradb-cli.

Changes

  • pyproject.toml — added license = { file = "LICENSE" } and classifiers (Development Status, Python versions, Apache Software License)

Testing

  • ruff check . — all checks passed
  • ruff format --check . — 13 files already formatted
  • python -c "import hydradb_deepeval" — import OK
  • No code changes, only metadata

- Add license = { file = "LICENSE" } field
- Add classifiers including Apache Software License
- Aligns metadata with hydradb-cli for org-wide consistency

Signed-off-by: Vorflux AI <noreply@vorflux.com>
@Aadil-5122 Aadil-5122 merged commit 3c05d40 into main Apr 8, 2026
5 checks passed
@greptile-apps
Copy link
Copy Markdown

greptile-apps bot commented Apr 8, 2026

Greptile Summary

This PR adds license = { file = "LICENSE" } and PyPI trove classifiers to pyproject.toml for metadata consistency with hydradb-cli. The LICENSE file (Apache 2.0) already exists in the repo and the declared classifier matches its contents — no code changes are introduced.

Confidence Score: 5/5

Safe to merge — metadata-only change with no code impact.

All findings are P2 style suggestions (missing Python 3.13 classifier, missing :: Only classifier). No code, logic, or runtime behaviour is touched; the LICENSE file exists and matches the declared classifier.

No files require special attention.

Vulnerabilities

No security concerns identified.

Important Files Changed

Filename Overview
pyproject.toml Adds license metadata (referencing existing Apache 2.0 LICENSE file) and PyPI trove classifiers; no code changes, LICENSE file present and matches declared classifier.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["pyproject.toml"] -->|"license = { file = 'LICENSE' }"| B["LICENSE (Apache 2.0)"]
    A -->|classifiers| C["PyPI Metadata"]
    C --> D["Development Status :: 4 - Beta"]
    C --> E["Programming Language :: Python :: 3.10/3.11/3.12"]
    C --> F["License :: OSI Approved :: Apache Software License"]
    C --> G["Intended Audience :: Developers"]
Loading

Reviews (1): Last reviewed commit: "chore: add license metadata and classifi..." | Re-trigger Greptile

Comment on lines +10 to +18
classifiers = [
"Development Status :: 4 - Beta",
"Programming Language :: Python :: 3",
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
"Programming Language :: Python :: 3.12",
"Intended Audience :: Developers",
"License :: OSI Approved :: Apache Software License",
]
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Python 3.13 missing from classifiers

requires-python = ">=3.10" implicitly covers Python 3.13 (and future releases), but the classifier list stops at 3.12. Users browsing PyPI will see no 3.13 badge even though the package is compatible. Consider adding "Programming Language :: Python :: 3.13" and "Programming Language :: Python :: 3 :: Only" to accurately reflect supported versions.

Suggested change
classifiers = [
"Development Status :: 4 - Beta",
"Programming Language :: Python :: 3",
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
"Programming Language :: Python :: 3.12",
"Intended Audience :: Developers",
"License :: OSI Approved :: Apache Software License",
]
classifiers = [
"Development Status :: 4 - Beta",
"Programming Language :: Python :: 3",
"Programming Language :: Python :: 3 :: Only",
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
"Programming Language :: Python :: 3.12",
"Programming Language :: Python :: 3.13",
"Intended Audience :: Developers",
"License :: OSI Approved :: Apache Software License",
]

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