Skip to content

Commit

Permalink
remove setup.cfg, build wheels from pyproject
Browse files Browse the repository at this point in the history
  • Loading branch information
aktech committed Aug 28, 2024
1 parent 9631a43 commit 5b4fead
Show file tree
Hide file tree
Showing 7 changed files with 18 additions and 15 deletions.
7 changes: 5 additions & 2 deletions .github/workflows/wheels.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,11 +80,14 @@ jobs:
bazel info
bazel version
- name: Install build
run: python -m pip install --upgrade pip build

- name: Build wheels
run: |
package_build/initialize.sh
python package_build/tfx/setup.py bdist_wheel
python package_build/ml-pipelines-sdk/setup.py bdist_wheel
python -m build --wheel package_build/tfx/
python -m build --wheel package_build/ml-pipelines-sdk/
mkdir wheelhouse
mv dist/*.whl wheelhouse/
Expand Down
1 change: 1 addition & 0 deletions package_build/initialize.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ do
ln -sf $BASEDIR/dist $BASEDIR/package_build/$CONFIG_NAME/
ln -sf $BASEDIR/tfx $BASEDIR/package_build/$CONFIG_NAME/
ln -sf $BASEDIR/README*.md $BASEDIR/package_build/$CONFIG_NAME/
ln -sf $BASEDIR/LICENSE $BASEDIR/package_build/$CONFIG_NAME/

rm -rf $BASEDIR/package_build/$CONFIG_NAME/build
mkdir $BASEDIR/package_build/$CONFIG_NAME/build
Expand Down
2 changes: 1 addition & 1 deletion package_build/ml-pipelines-sdk/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"

[project]
name = "ml-pipelines-sdk"
dynamic = ["version"]
dynamic = ["version", "dependencies", "optional-dependencies", "scripts"]
description = "A dependency-light distribution of the core pipeline authoring functionality of TensorFlow Extended (TFX)."
readme = "README.md"
license = { file = "LICENSE" }
Expand Down
2 changes: 1 addition & 1 deletion package_build/tfx/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"

[project]
name = "tfx"
dynamic = ["version"]
dynamic = ["version", "dependencies", "optional-dependencies", "scripts"]
description = "TensorFlow Extended (TFX) is a TensorFlow-based general-purpose machine learning platform implemented at Google."
readme = "README.md"
license = { file = "LICENSE" }
Expand Down
9 changes: 9 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -35,3 +35,12 @@ requires-python = ">=3.9,<3.11"
[project.urls]
Homepage = "https://www.tensorflow.org/tfx"
Repository = "https://github.com/tensorflow/tfx"

[tool.pytest.ini_options]
addopts = "--verbose -m 'not end_to_end'"
python_files = "*_test.py"
norecursedirs = ["custom_components", ".*", "*.egg"]
markers = [
"end_to_end: end-to-end tests which are slow and require more dependencies (deselect with '-m \"not end_to_end\"')",
"serial: mark tests that should not run in parallel"
]
10 changes: 0 additions & 10 deletions setup.cfg

This file was deleted.

2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ def finalize_options(self):
class _BuildCommand(build.build):
"""Build everything that is needed to install.
This overrides the original distutils "build" command to to run gen_proto
This overrides the original distutils "build" command to run gen_proto
command before any sub_commands.
build command is also invoked from bdist_wheel and install command, therefore
Expand Down

0 comments on commit 5b4fead

Please sign in to comment.