Skip to content

Conversation

@pauldambra
Copy link
Member

the release action is still failing https://github.com/PostHog/posthog-python/actions/runs/15468256593/job/43545576349

i can't run it locally

so, let's try this

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR addresses failing release actions by modernizing Python package configuration and removing deprecated license classifiers.

  • Consolidates configuration by migrating from setup.cfg to pyproject.toml following modern Python packaging standards
  • Removes deprecated License :: OSI Approved :: MIT License classifier from setup.py and setup_analytics.py while maintaining license declarations
  • Updates build settings in pyproject.toml with standardized line length configurations and pytest async testing support
  • Keep an eye on package discoverability on PyPI without the license classifier

4 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

classifiers=[
"Development Status :: 5 - Production/Stable",
"Intended Audience :: Developers",
"License :: OSI Approved :: MIT License",
Copy link
Member Author

Choose a reason for hiding this comment

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

i think maybe this is deprecated, let's move the license note to pyproject.toml and see if that helps

@pauldambra pauldambra requested a review from oliverb123 June 5, 2025 15:35
@pauldambra pauldambra changed the title fix: maybe the classifier is deprecated fix: python release attempt 3 Jun 6, 2025
@oliverb123 oliverb123 force-pushed the fix/release-try-2 branch from 097588e to c9bd525 Compare June 6, 2025 12:07
@oliverb123 oliverb123 merged commit 7af8e88 into master Jun 6, 2025
6 checks passed
@oliverb123 oliverb123 deleted the fix/release-try-2 branch June 6, 2025 12:16
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