Skip to content
This repository was archived by the owner on Aug 21, 2026. It is now read-only.

remove old rector - #714

Merged
mimmi20 merged 1 commit into
masterfrom
updates
Jul 23, 2026
Merged

remove old rector#714
mimmi20 merged 1 commit into
masterfrom
updates

Conversation

@mimmi20

@mimmi20 mimmi20 commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Refactor
    • Die Konfiguration für eine veraltete PHP-81-Regel wurde entfernt.
    • Alle übrigen Prüfungen und Regeln bleiben unverändert.

@mimmi20 mimmi20 self-assigned this Jul 23, 2026
@mimmi20 mimmi20 added the maintenance Maintenance label Jul 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c8c57512-7091-4383-bd80-c09d332fefda

📥 Commits

Reviewing files that changed from the base of the PR and between 270c9d6 and ea0ad55.

📒 Files selected for processing (1)
  • rector.php
💤 Files with no reviewable changes (1)
  • rector.php

📝 Walkthrough

Walkthrough

Die Rector-Konfiguration entfernt den Import und den Skip-Eintrag für FirstClassCallableRector. Dadurch ist diese Regel nicht mehr deaktiviert; die übrigen Rector-Regeln und Scopes bleiben unverändert.

Changes

Rector-Konfiguration

Layer / File(s) Summary
FirstClassCallable-Rector wieder aktivieren
rector.php
Der Import von FirstClassCallableRector und der entsprechende Eintrag in skip(...) werden entfernt. Die verbleibenden Konfigurationen bleiben bestehen.

Estimated code review effort: 1 (Trivial) | ~2 Minuten

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Der Titel passt zur Änderung: Eine alte Rector-Regel wurde aus der Konfiguration entfernt.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch updates

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.

@mimmi20
mimmi20 enabled auto-merge July 23, 2026 04:47
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (270c9d6) to head (ea0ad55).
⚠️ Report is 2 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##              master      #714   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        43        43           
===========================================
  Files              4         4           
  Lines            236       236           
===========================================
  Hits             236       236           
Flag Coverage Δ
php-8.3 100.00% <ø> (ø)
phpunit 100.00% <ø> (ø)
ubuntu-latest 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@mimmi20
mimmi20 merged commit 5cc49a8 into master Jul 23, 2026
64 checks passed
@mimmi20
mimmi20 deleted the updates branch July 23, 2026 04:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

maintenance Maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant