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

chore: [Gitar] Cleaning up stale feature flag: sdkReporting with value true #6717

Merged
merged 6 commits into from
Mar 28, 2024

Conversation

gitar-bot[bot]
Copy link
Contributor

@gitar-bot gitar-bot bot commented Mar 27, 2024

Feature Flag Cleanup
Stale Flag Value
sdkReporting true
Trigger

#6559 (comment)

Bot Commands

@gitar-bot cleanup stale_flag=value will cleanup a stale feature flag. Replace stale_flag with the name of the stale feature flag and value with either true or false.

@gitar-bot gitar-bot bot force-pushed the gitar_gitar__sdkReporting_true branch from 3a2d173 to 43c80fe Compare March 27, 2024 14:10
Copy link

vercel bot commented Mar 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
unleash-monorepo-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 28, 2024 7:07am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
unleash-docs ⬜️ Ignored (Inspect) Visit Preview Mar 28, 2024 7:07am

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

  • Declining Code Health: 0 findings(s) 🚩
  • Improving Code Health: 1 findings(s) ✅
  • Affected Hotspots: 0 files(s) 🔥

Recommended Review Level: Detailed -- Increased risk for defects: No code is modified, but the change set deletes (33) lines of code, which is an unusal pattern in this repo.
View detailed results in CodeScene

✅ Improving Code Health:

@@ -70,7 +68,6 @@ const StyledTab = styled(Tab)(({ theme }) => ({
}));

export const Application = () => {
const useOldApplicationScreen = !useUiFlag('sdkReporting');

Choose a reason for hiding this comment

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

✅ Getting better: Large Method
Application decreases from 139 to 135 lines of code, threshold = 120

Why does this problem occur?

Large functions with many lines of code are generally harder to understand and lower the code health. Avoid adding more lines to this function. Read more.

@sjaanus sjaanus self-assigned this Mar 27, 2024
Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

  • Declining Code Health: 0 findings(s) 🚩
  • Improving Code Health: 1 findings(s) ✅
  • Affected Hotspots: 0 files(s) 🔥

Recommended Review Level: Lightweight sanity check
View detailed results in CodeScene

✅ Improving Code Health:

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

  • Declining Code Health: 0 findings(s) 🚩
  • Improving Code Health: 1 findings(s) ✅
  • Affected Hotspots: 0 files(s) 🔥

Recommended Review Level: Lightweight sanity check
View detailed results in CodeScene

✅ Improving Code Health:

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

  • Declining Code Health: 0 findings(s) 🚩
  • Improving Code Health: 1 findings(s) ✅
  • Affected Hotspots: 0 files(s) 🔥

Recommended Review Level: Lightweight sanity check
View detailed results in CodeScene

✅ Improving Code Health:

@sjaanus sjaanus changed the title [Gitar] Cleaning up stale feature flag: sdkReporting with value true chore: [Gitar] Cleaning up stale feature flag: sdkReporting with value true Mar 27, 2024
Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

  • Declining Code Health: 0 findings(s) 🚩
  • Improving Code Health: 1 findings(s) ✅
  • Affected Hotspots: 0 files(s) 🔥

Recommended Review Level: Lightweight sanity check
View detailed results in CodeScene

✅ Improving Code Health:

@sjaanus sjaanus merged commit ca6946d into main Mar 28, 2024
14 checks passed
@sjaanus sjaanus deleted the gitar_gitar__sdkReporting_true branch March 28, 2024 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants