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

[Remix] Cleaning up stale feature flag: proPlanAutoCharge with value true #6581

Merged
merged 3 commits into from
Mar 15, 2024

Conversation

gitar-bot[bot]
Copy link
Contributor

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

Feature Flag Cleanup
Stale Flag Value
proPlanAutoCharge 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_remix__proPlanAutoCharge_true branch from 3f0587f to a8ed72c Compare March 15, 2024 09:07
Copy link

vercel bot commented Mar 15, 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 15, 2024 9:18am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
unleash-docs ⬜️ Ignored (Inspect) Visit Preview Mar 15, 2024 9:18am

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 (8) lines of code, which is an unusal pattern in this repo.
View detailed results in CodeScene

✅ Improving Code Health:

@@ -154,8 +154,7 @@ export const BillingPlan: FC<IBillingPlanProps> = ({ instanceStatus }) => {
</Grid>
<ConditionallyRender
condition={Boolean(
uiConfig?.flags?.proPlanAutoCharge &&
instanceStatus.plan === InstancePlan.PRO,
instanceStatus.plan === InstancePlan.PRO,

Choose a reason for hiding this comment

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

✅ Getting better: Large Method
BillingPlan:FC<IBillingPlanProps> decreases from 172 to 171 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.

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 merged commit 3d4aa4a into main Mar 15, 2024
14 checks passed
@sjaanus sjaanus deleted the gitar_remix__proPlanAutoCharge_true branch March 15, 2024 09:53
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

1 participant