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

feat(issue-states): add task to transition 14+ day old regressed issues to ONGOING #48042

Merged
merged 14 commits into from
May 1, 2023

Conversation

barkbarkimashark
Copy link
Contributor

@barkbarkimashark barkbarkimashark commented Apr 26, 2023

Nearly identical logic to #47881 just with some minor adjustments. Did some minor refactoring in this PR as well to account for the similarities.

Depends on https://github.com/getsentry/getsentry/pull/10403

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 26, 2023
@codecov
Copy link

codecov bot commented Apr 26, 2023

Codecov Report

Merging #48042 (dee33a5) into master (173d4a3) will decrease coverage by 0.01%.
The diff coverage is 89.18%.

❗ Current head dee33a5 differs from pull request most recent head 973fdb1. Consider uploading reports for the commit 973fdb1 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #48042      +/-   ##
==========================================
- Coverage   80.90%   80.89%   -0.01%     
==========================================
  Files        4772     4770       -2     
  Lines      201774   201696      -78     
  Branches    11525    11542      +17     
==========================================
- Hits       163239   163160      -79     
- Misses      38280    38281       +1     
  Partials      255      255              
Impacted Files Coverage Δ
src/sentry/conf/server.py 93.76% <ø> (ø)
src/sentry/tasks/auto_ongoing_issues.py 88.46% <84.00%> (-4.88%) ⬇️
src/sentry/features/__init__.py 100.00% <100.00%> (ø)
src/sentry/issues/ongoing.py 100.00% <100.00%> (ø)

... and 109 files with indirect coverage changes

Base automatically changed from sharky/issue-new-to-ongoing-auto to master April 27, 2023 22:06
@barkbarkimashark barkbarkimashark force-pushed the sharky/issue-regressed-to-ongoing-auto branch from 79e3c81 to 0dac6fc Compare April 27, 2023 22:53
@barkbarkimashark barkbarkimashark requested review from a team and snigdhas April 27, 2023 23:22
Copy link
Member

@snigdhas snigdhas left a comment

Choose a reason for hiding this comment

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

LGTM!

src/sentry/tasks/auto_ongoing_issues.py Outdated Show resolved Hide resolved
src/sentry/tasks/auto_ongoing_issues.py Outdated Show resolved Hide resolved
@barkbarkimashark barkbarkimashark enabled auto-merge (squash) May 1, 2023 22:55
@barkbarkimashark barkbarkimashark merged commit 20bdbfa into master May 1, 2023
@barkbarkimashark barkbarkimashark deleted the sharky/issue-regressed-to-ongoing-auto branch May 1, 2023 23:16
@armenzg armenzg added this to the Issue States and Filters milestone May 2, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jun 2, 2023
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants