Skip to content

fix: set correct dist#14

Merged
gao-sun merged 1 commit intomasterfrom
gao-fix-dist
May 8, 2025
Merged

fix: set correct dist#14
gao-sun merged 1 commit intomasterfrom
gao-fix-dist

Conversation

@gao-sun
Copy link
Copy Markdown
Contributor

@gao-sun gao-sun commented May 8, 2025

use find technique to include all source files

Testing

uv build now shows

adding 'mcpauth/__init__.py'
adding 'mcpauth/config.py'
adding 'mcpauth/exceptions.py'
adding 'mcpauth/types.py'
adding 'mcpauth/middleware/create_bearer_auth.py'
adding 'mcpauth/utils/__init__.py'
adding 'mcpauth/utils/_create_verify_jwt.py'
adding 'mcpauth/utils/_fetch_server_config.py'
adding 'mcpauth/utils/_validate_server_config.py'

@gao-sun gao-sun requested a review from Copilot May 8, 2025 17:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the package distribution settings by switching from the manual setuptools configuration to the find technique, ensuring all relevant source files are included.

  • Removed the old "[tool.setuptools]" configuration.
  • Added the "[tool.setuptools.packages.find]" configuration with "where" and "include" directives.

@gao-sun gao-sun merged commit b3c2cf7 into master May 8, 2025
5 checks passed
@gao-sun gao-sun deleted the gao-fix-dist branch May 8, 2025 17:34
@codecov
Copy link
Copy Markdown

codecov bot commented May 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

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.

2 participants