Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

docs(init): Fix sentry_sdk.init type hint #3283

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

szokeasaurusrex
Copy link
Member

The current type hint suggests that all the parameters can be passed as positional arguments, when this is not the case. Only the dsn can be passed as a positional argument; the rest must be passed as keyword arguments.

This PR makes the type hint reflect the reality of what parameters can be passed to sentry_sdk.init.

@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/fix-init-type-hint branch 2 times, most recently from 68e0c7e to 2d594c7 Compare July 12, 2024 14:31
The current type hint suggests that all the parameters can be passed as positional arguments, when this is not the case. Only the `dsn` can be passed as a positional argument; the rest must be passed as keyword arguments.

This PR makes the type hint reflect the reality of what parameters can be passed to `sentry_sdk.init`.
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/fix-init-type-hint branch from 2d594c7 to c97b2fb Compare July 12, 2024 14:49
@szokeasaurusrex szokeasaurusrex merged commit 8a95971 into master Jul 12, 2024
122 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/fix-init-type-hint branch July 12, 2024 15:02
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.

None yet

2 participants