Skip to content

ref: remove usage of isodate #93615

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

Merged
merged 1 commit into from
Jun 16, 2025
Merged

ref: remove usage of isodate #93615

merged 1 commit into from
Jun 16, 2025

Conversation

asottile-sentry
Copy link
Member

much simpler to just use seconds, only used in one place

much simpler to just use seconds, only used in one place
@asottile-sentry asottile-sentry requested a review from a team June 16, 2025 16:37
@asottile-sentry asottile-sentry requested review from a team as code owners June 16, 2025 16:37
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 16, 2025
@asottile-sentry asottile-sentry merged commit b85defc into master Jun 16, 2025
67 checks passed
@asottile-sentry asottile-sentry deleted the asottile-rm-isodate branch June 16, 2025 19:17
Copy link

sentry-io bot commented Jun 17, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ TypeError: unsupported type for timedelta seconds component: str sentry.api.helpers.deprecation in _should_be_bl... View Issue

Did you find this useful? React with a 👍 or 👎

billyvg pushed a commit that referenced this pull request Jun 18, 2025
much simpler to just use seconds, only used in one place

<!-- Describe your PR here. -->
andrewshie-sentry pushed a commit that referenced this pull request Jun 19, 2025
much simpler to just use seconds, only used in one place

<!-- Describe your PR here. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants