Fix tox error

[bjk7119]

Description

Fix tox error at github action

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Documentation update
• Refactoring, Maintenance
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Summary by CodeRabbit

• Chores
  • Optimized PyInstaller configuration for the build process.
  • Enhanced dependency handling by including additional package resources in the build.
  • Improved test discovery by excluding unnecessary directories from test execution.

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR modifies PyInstaller and test configurations. It simplifies cli.spec by removing path entries and reducing hidden imports, extends hook-fosslight_binary.py to include resources from additional packages (binaryornot, chardet), and adds pytest configuration to exclude specific directories from test discovery.

Changes

Cohort / File(s)	Summary
PyInstaller Configuration cli.spec, hooks/hook-fosslight_binary.py	Simplifies cli.spec by removing pathex entries and reducing hiddenimports; extends hook-fosslight_binary.py to collect data files, binaries, and hidden imports from binaryornot and chardet packages.
Test Configuration tox.ini	Adds pytest norecursedirs setting to exclude third_party, build, dist, .tox, .git, and \_\_pycache\_\_ directories from test discovery.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Fix tox error' is too vague and does not clearly describe the specific changes made (pytest configuration updates, PyInstaller hook modifications, and cli.spec adjustments).	Consider using a more descriptive title that indicates the main change, such as 'Update PyInstaller configuration and pytest settings' or 'Configure PyInstaller hooks for additional packages'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fixtox

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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