Skip to content
This repository was archived by the owner on Jul 16, 2025. It is now read-only.

Conversation

@Nikhil-Singhal-06
Copy link
Contributor

Description

find_namespace_packages: Use this when entry scripts are from other packages or when there's no init.py file in the entry script location folder.
find_packages: Use this when there are no entry points or external dependencies.

Pull request checklist

  • Issue created that explains the change and why it's needed
  • Tests are part of the PR (for bug fixes / features)
  • Docs reviewed and added / updated if needed (for bug fixes / features)
  • Format and Lint checks (make check) pass locally
  • PR applies Apache 2.0 License to all code files

Pull request type

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation changes
  • Other (please describe)

Current behavior

Python build warnings "library is not found."

New behavior

Clean build

How to test

Colcon build

@fred-labs fred-labs self-requested a review August 12, 2024 11:23
@fred-labs fred-labs enabled auto-merge (squash) August 12, 2024 11:23
@github-actions
Copy link

Test Results

266 tests   251 ✅  38m 47s ⏱️
 16 suites   15 💤
 16 files      0 ❌

Results for commit 10ff437.

@fred-labs fred-labs merged commit e50339c into main Aug 12, 2024
@fred-labs fred-labs deleted the fix_warnings branch August 12, 2024 11:24
fred-labs pushed a commit that referenced this pull request Aug 13, 2024
fred-labs pushed a commit that referenced this pull request Aug 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants