Skip to content

Make use of ExceptionGroup when autodetection of style fails.#79

Draft
JanEricNitschke wants to merge 3 commits intomainfrom
exception_groups
Draft

Make use of ExceptionGroup when autodetection of style fails.#79
JanEricNitschke wants to merge 3 commits intomainfrom
exception_groups

Conversation

@JanEricNitschke
Copy link
Copy Markdown
Owner

@JanEricNitschke JanEricNitschke commented Mar 19, 2024

Fixes #76

Can be used once 3.9 and 3.10 become EOL in late 2026

@JanEricNitschke JanEricNitschke requested a review from Copilot April 11, 2025 09:00
Copy link
Copy Markdown

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.

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

pymend/docstring_parser/parser.py:41

  • There is a typo in the docstring; consider revising 'module an parse' to 'modules that can parse'.
If none of the available module an parse the docstring

Copy link
Copy Markdown
Owner Author

@JanEricNitschke JanEricNitschke left a comment

Choose a reason for hiding this comment

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

Should use more specific syntax in the test:

https://docs.pytest.org/en/stable/how-to/assert.html#assertions-about-expected-exception-groups

with pytest.RaisesGroup(ValueError):

@JanEricNitschke JanEricNitschke marked this pull request as draft July 18, 2025 09:58
@JanEricNitschke JanEricNitschke force-pushed the exception_groups branch 4 times, most recently from 01c9edb to a1d2a0e Compare July 20, 2025 18:06
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.

Raise exceptiongroup in parser for auto style detection

2 participants