Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(event): don't attempt to import exceptiongroup if running python 3.11+ #525

Merged
merged 2 commits into from
Jul 1, 2024

Conversation

actionless
Copy link
Contributor

don't attempt to import exceptiongroup if running python 3.11+

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Documentation content changes

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@danyeaw danyeaw merged commit 378009e into gaphor:main Jul 1, 2024
3 checks passed
@danyeaw
Copy link
Member

danyeaw commented Jul 1, 2024

Thanks @actionless, much appreciated!

@actionless
Copy link
Contributor Author

i just realized i forgot to make it to be a conditional dependency in the pyproject (https://peps.python.org/pep-0631/)

i'll make a separate PR when i'll have a free minute

@actionless actionless deleted the patch-1 branch July 2, 2024 13:02
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants