Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion sgl-kernel/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ build-backend = "scikit_build_core.build"

[project]
name = "sgl-kernel"
version = "0.3.18.post2"
version = "0.3.18.post3"
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

To improve maintainability and establish a single source of truth for the version number, consider using dynamic versioning. This would allow the version to be read directly from python/sgl_kernel/version.py, eliminating the need for manual updates in this file.

You can implement this by replacing this line with dynamic = ["version"] and adding the following to your [tool.scikit-build] section:

metadata.version.path = "python/sgl_kernel/version.py"

scikit-build-core defaults to a regex provider that will correctly parse the __version__ variable. This change will make python/sgl_kernel/version.py the single source of truth, simplifying the version bumping process.

Suggested change
version = "0.3.18.post3"
dynamic = ["version"]

authors = [
{ name="Yineng Zhang", email="me@zhyncs.com" },
]
Expand Down
2 changes: 1 addition & 1 deletion sgl-kernel/pyproject_cpu.toml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ build-backend = "scikit_build_core.build"

[project]
name = "sgl-kernel"
version = "0.3.18.post2"
version = "0.3.18.post3"
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

To improve maintainability and establish a single source of truth for the version number, consider using dynamic versioning. This would allow the version to be read directly from python/sgl_kernel/version.py, eliminating the need for manual updates in this file.

You can implement this by replacing this line with dynamic = ["version"] and adding the following to your [tool.scikit-build] section:

metadata.version.path = "python/sgl_kernel/version.py"

scikit-build-core defaults to a regex provider that will correctly parse the __version__ variable. This change will make python/sgl_kernel/version.py the single source of truth, simplifying the version bumping process.

Suggested change
version = "0.3.18.post3"
dynamic = ["version"]

description = "Kernel Library for SGLang"
readme = "README.md"
requires-python = ">=3.10"
Expand Down
2 changes: 1 addition & 1 deletion sgl-kernel/pyproject_rocm.toml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ build-backend = "setuptools.build_meta"

[project]
name = "sgl-kernel"
version = "0.3.18.post2"
version = "0.3.18.post3"
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

To improve maintainability and establish a single source of truth for the version number, consider using dynamic versioning. This would allow the version to be read directly from python/sgl_kernel/version.py, eliminating manual updates in this file.

Since this file uses setuptools as the build backend, you can achieve this by replacing this line with dynamic = ["version"] and adding a [tool.setuptools] section to configure it:

[tool.setuptools]
package-dir = {"" = "python"}

[tool.setuptools.dynamic]
version = { attr = "sgl_kernel.version.__version__" }

This change will make python/sgl_kernel/version.py the single source of truth and simplify the version bumping process.

Suggested change
version = "0.3.18.post3"
dynamic = ["version"]

description = "Kernel Library for SGLang"
readme = "README.md"
requires-python = ">=3.10"
Expand Down
2 changes: 1 addition & 1 deletion sgl-kernel/python/sgl_kernel/version.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
__version__ = "0.3.18.post2"
__version__ = "0.3.18.post3"
Loading