phalt / graphene-django-sentry

Capture Sentry exceptions through django-graphene views
MIT License
22 stars 6 forks source link

setup.py: Relax `sentry-sdk` constraint? #9

Open tony opened 1 week ago

tony commented 1 week ago

Description

Installing via uv (repo, docs, PyPI) ran into the sentry-sdk constraint in setup.py: https://github.com/phalt/graphene-django-sentry/blob/49e05345d84605ddad7cec36da7410a3100658ec/setup.py#L13

  × No solution found when resolving dependencies:
  ╰─▶ Because graphene-sentry==0.4.0 depends on sentry-sdk>=0.5.2,<=0.10.1 and only graphene-sentry==0.4.0 is available, we can conclude that all versions of graphene-sentry depend on sentry-sdk>=0.5.2,<=0.10.1.
      And because your project depends on graphene-sentry and sentry-sdk>=2.14.0,<2.15, we can conclude that your project's requirements are unsatisfiable.

What I Did

uv 0.5.1 with these dependencies:

[project]
dependencies = [
   "python >=3.12,<4",
  "graphene-sentry >=0.4.0,<0.5",
  "sentry-sdk >=2.14.0,<2.15",
]
tony commented 1 week ago

@phalt Forgive me! I got the wrong repository - I intended to write in graphene-sentry (PyPI)

However, it is a fork of your repo and has the same issue.

The solution would be to remove the sentry-sdk constraint from setup.py, so instead of:

requirements = [
    'graphene_django==2.2.0',
    'sentry-sdk==0.5.2',
]

Change it to:

requirements = [
    'graphene_django==2.2.0',
    'sentry-sdk',
]

Suggestions (from a downstream perspective):

tony commented 1 week ago

Also, thank you for the project! 👋