Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Sep 10, 2025

fix: exclude tests and scripts directories from package build

Summary

Added exclude = ["tests*", "scripts*"] to the [tool.setuptools.packages.find] section in pyproject.toml to prevent test and script directories from being included in the built Python wheel package. This follows Python packaging best practices and reduces the package size by excluding development-only files from distribution.

Review & Testing Checklist for Human

  • Verify that python -m build --wheel produces a wheel that excludes tests and scripts directories
  • Test installing the built package in a clean environment to ensure functionality is preserved
  • Confirm that existing unit tests still run properly with pytest tests/unit

Notes

This change implements the exact configuration requested and has been verified to work correctly - the build process now excludes tests and scripts while preserving all core functionality. Unit tests continue to pass, confirming the change doesn't affect development workflows.

Link to Devin run: https://app.devin.ai/sessions/130d75c2f4c44bc8b8b180fa78608951
Requested by: daniel@genlayerlabs.com (@danielrc888)

@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@coderabbitai
Copy link

coderabbitai bot commented Sep 10, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@danieljrc888 danieljrc888 self-assigned this Sep 10, 2025
@danieljrc888 danieljrc888 changed the title build: exclude tests and scripts directories from package build perf: exclude tests and scripts directories from package build Sep 10, 2025
@danieljrc888 danieljrc888 self-requested a review September 10, 2025 09:21
@danieljrc888 danieljrc888 force-pushed the devin/1757495658-exclude-tests-scripts-from-build branch 3 times, most recently from f7323aa to c88ab2d Compare September 10, 2025 09:28
- Add exclude configuration to [tool.setuptools.packages.find] section
- Prevents tests* and scripts* directories from being included in wheel packages
- Reduces package size and follows best practices for distribution
- Verified build process works correctly and tests still run

Co-Authored-By: daniel@genlayerlabs.com <danielrojasc888@gmail.com>
@danieljrc888 danieljrc888 force-pushed the devin/1757495658-exclude-tests-scripts-from-build branch from c88ab2d to c03fc72 Compare September 10, 2025 09:29
@danieljrc888 danieljrc888 changed the title perf: exclude tests and scripts directories from package build fix: exclude tests and scripts directories from package build Sep 10, 2025
@danieljrc888 danieljrc888 requested review from cristiam86 and removed request for danieljrc888 September 10, 2025 09:29
@cristiam86 cristiam86 merged commit 51986f0 into main Sep 10, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants