Skip to content

Version 7.6.0#4429

Merged
ildyria merged 2 commits into
masterfrom
v8-gate
Jun 17, 2026
Merged

Version 7.6.0#4429
ildyria merged 2 commits into
masterfrom
v8-gate

Conversation

@ildyria

@ildyria ildyria commented Jun 17, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features

    • Added V8 feature support with configurable environment toggle to enable or disable functionality.
  • Chores

    • Version bumped to 7.6.0.

@ildyria ildyria requested a review from a team as a code owner June 17, 2026 19:06
@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a V8_ENABLED environment-backed feature flag to config/features.php. In SettingsController, a new public const V8_CONFIGS extracts the V8-related config key list, and getAll() references it for conditional filtering. All three PHPUnit configs gain V8_ENABLED=true. A new migration bumps the configs table version to 070600 (rolling back to 070504), and version.md is updated to 7.6.0.

Changes

V8 Feature Flag and Settings Filtering

Layer / File(s) Summary
V8 feature flag config and SettingsController filtering
config/features.php, app/Http/Controllers/Admin/SettingsController.php
Adds 'v8' => (bool) env('V8_ENABLED', false) to the features config. Extracts V8 config key names into public const V8_CONFIGS in SettingsController and updates getAll() to reference self::V8_CONFIGS when the flag is disabled.
PHPUnit V8_ENABLED env var
phpunit.xml, phpunit.ci.xml, phpunit.pgsql.xml
Adds V8_ENABLED=true to all three PHPUnit XML configurations so tests run with the V8 flag active.

Version Bump to 7.6.0

Layer / File(s) Summary
Migration and version.md bump to 7.6.0
database/migrations/2026_06_17_190527_bump_version070600.php, version.md
New migration updates configs.version to 070600, clears application cache with exception handling and console output, and reverts to 070504 in down(). version.md changed from 7.5.4 to 7.6.0.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐇 Hop hop, the version leaps to seven-six-oh,
A flag for V8, now the features grow!
The constant extracted, no more inline sprawl,
Cache cleared on migration, standing tall.
V8_ENABLED=true in every test file neat —
This bunny's refactor is oh-so sweet! 🥕

🚥 Pre-merge checks | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 75.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 10773096-0cc5-43ea-9e5a-dfd74d3c6c8e

📥 Commits

Reviewing files that changed from the base of the PR and between 6e1e430 and b9fe948.

📒 Files selected for processing (7)
  • app/Http/Controllers/Admin/SettingsController.php
  • config/features.php
  • database/migrations/2026_06_17_190527_bump_version070600.php
  • phpunit.ci.xml
  • phpunit.pgsql.xml
  • phpunit.xml
  • version.md

Comment thread version.md
@ildyria ildyria merged commit 69042ec into master Jun 17, 2026
42 checks passed
@ildyria ildyria deleted the v8-gate branch June 17, 2026 19:55
@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.47%. Comparing base (6e1e430) to head (b9fe948).
⚠️ Report is 1 commits behind head on master.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant