Address code review feedback on comments and documentation

Copilot · johnslavik · johnslavik · commit bb98dcaf0bb9 · 2026-01-18T12:41:52.000+01:00
Co-authored-by: johnslavik &lt;64036239+johnslavik@users.noreply.github.com&gt;
diff --git a/packages/griffecli/pyproject.toml b/packages/griffecli/pyproject.toml
@@ -45,9 +45,11 @@ classifiers = [
 griffecli = "griffecli:main"
 
 [tool.hatch.metadata.hooks.uv-dynamic-versioning]
+# Dependencies are dynamically versioned; {version} is substituted at build time.
 dependencies = ["griffelib=={version}", "colorama>=0.4"]
 
 [tool.hatch.metadata.hooks.uv-dynamic-versioning.optional-dependencies]
+# No optional dependencies for the CLI package
 
 [tool.hatch.build.targets.sdist]
 
diff --git a/packages/griffecli/src/griffecli/_internal/__init__.py b/packages/griffecli/src/griffecli/_internal/__init__.py
@@ -1 +1 @@
-# Internal module for griffecli.
+# Internal modules for the griffecli package.
diff --git a/packages/griffelib/pyproject.toml b/packages/griffelib/pyproject.toml
@@ -42,8 +42,11 @@ classifiers = [
 ]
 
 [tool.hatch.metadata.hooks.uv-dynamic-versioning]
+# No base dependencies needed for griffelib
 
 [tool.hatch.metadata.hooks.uv-dynamic-versioning.optional-dependencies]
+# The 'pypi' extra provides dependencies needed for the load_pypi functionality
+# to download and inspect packages from PyPI.
 pypi = ["pip>=24.0", "platformdirs>=4.2", "wheel>=0.42"]
 
 [tool.hatch.build.targets.sdist]

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-# Internal module for griffecli.`
	`1`	`+# Internal modules for the griffecli package.`
Original file line number	Diff line number	Diff line change
`@@ -42,8 +42,11 @@ classifiers = [`
`42`	`42`	`]`
`43`	`43`
`44`	`44`	`[tool.hatch.metadata.hooks.uv-dynamic-versioning]`
	`45`	`+# No base dependencies needed for griffelib`
`45`	`46`
`46`	`47`	`[tool.hatch.metadata.hooks.uv-dynamic-versioning.optional-dependencies]`
	`48`	`+# The 'pypi' extra provides dependencies needed for the load_pypi functionality`
	`49`	`+# to download and inspect packages from PyPI.`
`47`	`50`	`pypi = ["pip>=24.0", "platformdirs>=4.2", "wheel>=0.42"]`
`48`	`51`
`49`	`52`	`[tool.hatch.build.targets.sdist]`