Commit 3b499f9a authored by Adam Hegyi's avatar Adam Hegyi

Eliminate FF for first_mentioned_in_commit_at

This commit cleans up a feature flag called:
`store_first_mentioned_in_commit_on_issue_close`
parent e3a619f1
...@@ -59,8 +59,6 @@ module Issues ...@@ -59,8 +59,6 @@ module Issues
end end
def store_first_mentioned_in_commit_at(issue, merge_request) def store_first_mentioned_in_commit_at(issue, merge_request)
return unless Feature.enabled?(:store_first_mentioned_in_commit_on_issue_close, issue.project, default_enabled: true)
metrics = issue.metrics metrics = issue.metrics
return if metrics.nil? || metrics.first_mentioned_in_commit_at return if metrics.nil? || metrics.first_mentioned_in_commit_at
......
...@@ -163,19 +163,6 @@ describe Issues::CloseService do ...@@ -163,19 +163,6 @@ describe Issues::CloseService do
expect(issue.metrics.first_mentioned_in_commit_at).to be_nil expect(issue.metrics.first_mentioned_in_commit_at).to be_nil
end end
end end
context 'when `store_first_mentioned_in_commit_on_issue_close` feature flag is off' do
before do
stub_feature_flags(store_first_mentioned_in_commit_on_issue_close: { enabled: false, thing: issue.project })
end
it 'does not update the metrics' do
subject
expect(described_class).not_to receive(:store_first_mentioned_in_commit_at)
expect(issue.metrics.first_mentioned_in_commit_at).to be_nil
end
end
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment