Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add a schema for validate-pyproject #447

Merged
merged 2 commits into from
Aug 2, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
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
24 changes: 24 additions & 0 deletions docs/api/scikit_build_core.settings.rst
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,22 @@ scikit\_build\_core.settings package
Submodules
----------

scikit\_build\_core.settings.documentation module
-------------------------------------------------

.. automodule:: scikit_build_core.settings.documentation
:members:
:undoc-members:
:show-inheritance:

scikit\_build\_core.settings.json\_schema module
------------------------------------------------

.. automodule:: scikit_build_core.settings.json_schema
:members:
:undoc-members:
:show-inheritance:

scikit\_build\_core.settings.metadata module
--------------------------------------------

Expand All @@ -33,6 +49,14 @@ scikit\_build\_core.settings.skbuild\_read\_settings module
:undoc-members:
:show-inheritance:

scikit\_build\_core.settings.skbuild\_schema module
---------------------------------------------------

.. automodule:: scikit_build_core.settings.skbuild_schema
:members:
:undoc-members:
:show-inheritance:

scikit\_build\_core.settings.sources module
-------------------------------------------

Expand Down
1 change: 1 addition & 0 deletions docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@
"sphinx_copybutton",
"sphinx_inline_tabs",
"conftabs",
"sphinx-jsonschema",
]

# Add any paths that contain templates here, relative to this directory.
Expand Down
8 changes: 8 additions & 0 deletions docs/configuration.md
Original file line number Diff line number Diff line change
Expand Up @@ -452,3 +452,11 @@ only be used if you enable them:
[tool.scikit-build]
experimental = true
```

## Full schema

The full schema for the `tool.scikit-build` table is below:

```{jsonschema} ../src/scikit_build_core/resources/toml_schema.json

```
2 changes: 1 addition & 1 deletion docs/examples/downstream/nanobind_example/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ authors = [
{ name = "Wenzel Jakob", email = "wenzel.jakob@epfl.ch" },
]
classifiers = [
"License :: BSD",
"License :: OSI Approved :: BSD License",
]

[project.urls]
Expand Down
20 changes: 20 additions & 0 deletions noxfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,26 @@ def _run_tests(
session.run("pytest", *run_args, *posargs, env=env)


@nox.session(reuse_venv=True)
def generate_schema(session: nox.Session) -> None:
session.install("-e.")
schema = session.run(
"python", "-m", "scikit_build_core.settings.skbuild_schema", silent=True
)
DIR.joinpath("src/scikit_build_core/resources/toml_schema.json").write_text(schema) # type: ignore[arg-type]


@nox.session
def validate_schemas(session: nox.Session) -> None:
session.install("-e.", "validate-pyproject")

docs_examples = Path("docs/examples").glob("**/pyproject.toml")
tests_packages = Path("tests/packages").glob("**/pyproject.toml")
paths = [*docs_examples, *tests_packages]
# mop currently required to make mypy happy, future nox version should fix
henryiii marked this conversation as resolved.
Show resolved Hide resolved
session.run("validate-pyproject", *map(str, paths))


@nox.session
def tests(session: nox.Session) -> None:
"""
Expand Down
2 changes: 2 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,7 @@ docs = [
"sphinx >=4.0",
"sphinx-copybutton",
"sphinx-inline-tabs",
"sphinx-jsonschema",
]

[project.urls]
Expand All @@ -102,6 +103,7 @@ Issues = "https://github.com/scikit-build/scikit-build-core/issues"
"distutils.setup_keywords".cmake_source_dir = "scikit_build_core.setuptools.build_cmake:cmake_source_dir"
"distutils.setup_keywords".cmake_args = "scikit_build_core.setuptools.build_cmake:cmake_args"
"setuptools.finalize_distribution_options".scikit_build_entry = "scikit_build_core.setuptools.build_cmake:finalize_distribution_options"
"validate_pyproject.tool_schema".scikit-build = "scikit_build_core.settings.skbuild_schema:get_skbuild_schema"

[tool.hatch]
version.source = "vcs"
Expand Down
221 changes: 221 additions & 0 deletions src/scikit_build_core/resources/toml_schema.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,221 @@
{
"$schema": "http://json-schema.org/draft-07/schema",
"$id": "https://github.com/scikit-build/scikit-build/blob/main/src/scikit_build_core/resources/toml_schema.json",
"description": "Scikit-build-core's settings.",
"type": "object",
"additionalProperties": false,
"properties": {
"cmake": {
"type": "object",
"additionalProperties": false,
"properties": {
"minimum-version": {
"type": "string",
"default": "3.15",
"description": "The minimum version of CMake to use. If CMake is older than this, it will be upgraded via PyPI if possible. An empty string will disable this check."
},
"args": {
"type": "array",
"items": {
"type": "string"
},
"description": "A list of args to pass to CMake when configuring the project."
},
"define": {
"type": "object",
"patternProperties": {
".+": {
"type": "string"
}
},
"description": "A table of defines to pass to CMake when configuring the project. Additive."
},
"verbose": {
"type": "boolean",
"default": false,
"description": "Verbose printout when building"
},
"build-type": {
"type": "string",
"default": "Release",
"description": "The build type to use when building the project. Valid options are: \"Debug\", \"Release\", \"RelWithDebInfo\", \"MinSizeRel\", \"\", etc."
},
"source-dir": {
"type": "string",
"default": "",
"description": "The source directory to use when building the project. Currently only affects the native builder (not the setuptools plugin)."
},
"targets": {
"type": "array",
"items": {
"type": "string"
},
"description": "The build targets to use when building the project. Empty builds the default target."
}
}
},
"ninja": {
"type": "object",
"additionalProperties": false,
"properties": {
"minimum-version": {
"type": "string",
"default": "1.5",
"description": "The minimum version of Ninja to use. If Ninja is older than this, it will be upgraded via PyPI if possible. An empty string will disable this check."
},
"make-fallback": {
"type": "boolean",
"default": true,
"description": "If make is present, do not add ninja if missing."
}
}
},
"logging": {
"type": "object",
"additionalProperties": false,
"properties": {
"level": {
"type": "string",
"default": "WARNING",
"description": "The logging level to display."
}
}
},
"sdist": {
"type": "object",
"additionalProperties": false,
"properties": {
"include": {
"type": "array",
"items": {
"type": "string"
},
"description": "Files to include in the SDist even if they are skipped by default."
},
"exclude": {
"type": "array",
"items": {
"type": "string"
},
"description": "Files to exclude from the SDist even if they are included by default."
},
"reproducible": {
"type": "boolean",
"default": true,
"description": "If set to True, try to build a reproducible distribution. ``SOURCE_DATE_EPOCH`` will be used for timestamps, or a fixed value if not set."
}
}
},
"wheel": {
"type": "object",
"additionalProperties": false,
"properties": {
"packages": {
"type": "array",
"items": {
"type": "string"
},
"description": "A list of packages to auto-copy into the wheel. If this is None, it will default to the first of ``src/<package>`` or ``<package>`` if they exist. The prefix(s) will be stripped from the package name inside the wheel."
},
"py-api": {
"type": "string",
"default": "",
"description": "The Python tags. The default (empty string) will use the default Python version. You can also set this to \"cp37\" to enable the CPython 3.7+ Stable ABI / Limited API (only on CPython and if the version is sufficient, otherwise this has no effect). Or you can set it to \"py3\" or \"py2.py3\" to ignore Python ABI compatibility. For the stable ABI, the CMake variable SKBUILD_SOABI will be set to abi3 on Unix-like systems (empty on Windows). FindPython doesn't have a way to target python3.dll instead of python3N.dll, so this is harder to use on Windows. The ABI tag is inferred from this tag."
},
"expand-macos-universal-tags": {
"type": "boolean",
"default": false,
"description": "Fill out extra tags that are not required. This adds \"x86_64\" and \"arm64\" to the list of platforms when \"universal2\" is used, which helps older Pip's (before 21.0.1) find the correct wheel."
},
"install-dir": {
"type": "string",
"default": "",
"description": "The install directory for the wheel. This is relative to the platlib root. EXPERIMENTAL: An absolute path will be one level higher than the platlib root, giving access to \"/platlib\", \"/data\", \"/headers\", and \"/scripts\"."
},
"license-files": {
"type": "array",
"items": {
"type": "string"
},
"description": "A list of license files to include in the wheel. Supports glob patterns."
}
}
},
"backport": {
"type": "object",
"additionalProperties": false,
"properties": {
"find-python": {
"type": "string",
"default": "3.26.1",
"description": "If CMake is less than this value, backport a copy of FindPython. Set to 0 disable this, or the empty string."
}
}
},
"metadata": {
"type": "object",
"patternProperties": {
".+": {
"type": "object"
}
}
},
"editable": {
"type": "object",
"additionalProperties": false,
"properties": {
"mode": {
"type": "string",
"default": "redirect",
"description": "Select the editable mode to use. Currently only \"redirect\" is supported."
},
"verbose": {
"type": "boolean",
"default": true,
"description": "Turn on verbose output for the editable mode rebuilds."
},
"rebuild": {
"type": "boolean",
"default": false,
"description": "Rebuild the project when the package is imported. The build-directory must be set."
}
}
},
"install": {
"type": "object",
"additionalProperties": false,
"properties": {
"components": {
"type": "array",
"items": {
"type": "string"
},
"description": "The components to install. If empty, the default is used."
},
"strip": {
"type": "boolean",
"description": "Whether to strip the binaries. True for scikit-build-core 0.5+."
}
}
},
"strict-config": {
"type": "boolean",
"default": true,
"description": "Strictly check all config options. If False, warnings will be printed for unknown options. If True, an error will be raised."
},
"experimental": {
"type": "boolean",
"default": false,
"description": "Enable early previews of features not finalized yet."
},
"minimum-version": {
"type": "string",
"description": "If set, this will provide a method for backward compatibility."
},
"build-dir": {
"type": "string",
"default": "",
"description": "The build directory. Defaults to a temporary directory, but can be set."
}
}
}
36 changes: 36 additions & 0 deletions src/scikit_build_core/settings/documentation.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
from __future__ import annotations

import ast
import inspect
import sys
import textwrap

__all__ = ["pull_docs"]


def __dir__() -> list[str]:
return __all__


def _get_value(value: ast.expr) -> str:
if sys.version_info < (3, 8):
assert isinstance(value, ast.Str)
return value.s

assert isinstance(value, ast.Constant)
return value.value


def pull_docs(dc: type[object]) -> dict[str, str]:
"""
Pulls documentation from a dataclass.
"""
t = ast.parse(inspect.getsource(dc))
(obody,) = t.body
assert isinstance(obody, ast.ClassDef)
body = obody.body
return {
assign.target.id: textwrap.dedent(_get_value(expr.value)).strip().replace("\n", " ") # type: ignore[union-attr]
for assign, expr in zip(body[:-1], body[1:])
if isinstance(assign, ast.AnnAssign) and isinstance(expr, ast.Expr)
}
Loading
Loading