Skip to content

Create stub for root pyproject editable build#6643

Merged
StafaH merged 1 commit into
isaac-sim:developfrom
StafaH:mh/root_package_hack
Jul 21, 2026
Merged

Create stub for root pyproject editable build#6643
StafaH merged 1 commit into
isaac-sim:developfrom
StafaH:mh/root_package_hack

Conversation

@StafaH

@StafaH StafaH commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Description

Create a stub so that the root pyproject can be treated as a package so users can editable install the test package bundle.

Checklist

  • I have read and understood the contribution guidelines
  • I have run the pre-commit checks with ./isaaclab.sh --format
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the changelog and the corresponding version in the extension's config/extension.toml file
  • I have added my name to the CONTRIBUTORS.md or my name already exists there

@StafaH
StafaH requested a review from a team July 21, 2026 03:59
@greptile-apps

greptile-apps Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR makes the repository root available as an empty editable package. The main changes are:

  • Adds a setuptools PEP 517 build backend.
  • Disables package and module discovery for the root distribution.

Confidence Score: 5/5

This looks safe to merge.

  • The empty package configuration matches the stated purpose.
  • No concrete build, runtime, or security issue was found in the changed code.

Important Files Changed

Filename Overview
pyproject.toml Adds an intentionally empty setuptools build configuration for editable root installs.

Reviews (1): Last reviewed commit: "Fix issue with pip install for test" | Re-trigger Greptile

@StafaH
StafaH merged commit 776aa2c into isaac-sim:develop Jul 21, 2026
39 checks passed
@StafaH
StafaH deleted the mh/root_package_hack branch July 23, 2026 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants