Skip to content

Conversation

@asottile-sentry
Copy link
Contributor

this setting is required by dmypy and needed to auto type things

@asottile-sentry asottile-sentry requested review from a team April 17, 2025 13:42
@asottile-sentry asottile-sentry requested review from a team as code owners April 17, 2025 13:42
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 17, 2025
@codecov
Copy link

codecov bot commented Apr 17, 2025

Codecov Report

Attention: Patch coverage is 92.85714% with 7 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/filestore/gcs.py 12.50% 7 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #89854       +/-   ##
===========================================
+ Coverage   63.44%   87.70%   +24.25%     
===========================================
  Files       10204    10206        +2     
  Lines      575452   575449        -3     
  Branches    22677    22677               
===========================================
+ Hits       365106   504682   +139576     
+ Misses     209910    70331   -139579     
  Partials      436      436               

this setting is required by dmypy and needed to auto type things
@asottile-sentry asottile-sentry merged commit ed98fca into master Apr 18, 2025
64 checks passed
@asottile-sentry asottile-sentry deleted the asottile-partial-types branch April 18, 2025 13:19
@asottile-sentry asottile-sentry added the Trigger: Revert Add to a merged PR to revert it (skips CI) label Apr 18, 2025
@getsentry-bot
Copy link
Contributor

PR reverted: 5198907

getsentry-bot added a commit that referenced this pull request Apr 18, 2025
This reverts commit ed98fca.

Co-authored-by: asottile-sentry <[email protected]>
@asottile-sentry
Copy link
Contributor Author

looks like our production configuration is broken -- will try again after I fix that:

TypeError: GoogleCloudStorage.__init__() got an unexpected keyword argument 'project'. Did you mean 'project_id'?

asottile-sentry added a commit that referenced this pull request Apr 21, 2025
asottile-sentry added a commit that referenced this pull request Apr 22, 2025
first commit automated via
https://github.com/getsentry/auto-type-annotate

- I used these patches to mypy:
    - python/mypy#18948
- (the line mentioned in the issue commented out):
python/mypy#18940 (comment)
- and with this unreverted (coming soon!):
#89854

<!-- Describe your PR here. -->
andrewshie-sentry pushed a commit that referenced this pull request Apr 22, 2025
this setting is required by dmypy and needed to auto type things

<!-- Describe your PR here. -->
andrewshie-sentry pushed a commit that referenced this pull request Apr 22, 2025
This reverts commit ed98fca.

Co-authored-by: asottile-sentry <[email protected]>
andrewshie-sentry pushed a commit that referenced this pull request Apr 22, 2025
first commit automated via
https://github.com/getsentry/auto-type-annotate

- I used these patches to mypy:
    - python/mypy#18948
- (the line mentioned in the issue commented out):
python/mypy#18940 (comment)
- and with this unreverted (coming soon!):
#89854

<!-- Describe your PR here. -->
asottile-sentry added a commit that referenced this pull request Apr 23, 2025
This reverts commit 5198907.

clean reapply -- I've fixed the filestore options in production via
https://github.com/getsentry/getsentry/pull/17256 and
https://github.com/getsentry/getsentry/pull/17262

<!-- Describe your PR here. -->
getsentry-bot added a commit that referenced this pull request Apr 24, 2025
markstory added a commit that referenced this pull request Apr 24, 2025
…" (#90001)""

This reverts commit 89cf46b. This
commit was reverted as part of an incident as it went out in a suspect
release, but this change was not a contributing factor.
markstory added a commit that referenced this pull request Apr 24, 2025
…" (#90001)"" (#90272)

This reverts commit 89cf46b. This
commit was reverted as part of an incident as it went out in a suspect
release, but this change was not a contributing factor.

Refs #90001
@github-actions github-actions bot locked and limited conversation to collaborators May 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components Trigger: Revert Add to a merged PR to revert it (skips CI)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants