Skip to content

Feat(Build) add REctor #1029

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Oct 10, 2025
Merged

Feat(Build) add REctor #1029

merged 9 commits into from
Oct 10, 2025

Conversation

@stonebuzz
Copy link
Contributor

@stonebuzz stonebuzz commented Oct 10, 2025

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

Add Rector

Screenshots (if appropriate):

@stonebuzz stonebuzz requested a review from Rom1-B October 10, 2025 09:19
@stonebuzz stonebuzz self-assigned this Oct 10, 2025
Rom1-B
Rom1-B requested changes Oct 10, 2025
View reviewed changes
stonebuzz and others added 5 commits October 10, 2025 11:49
@stonebuzz @Rom1-B
Update ajax/container.php
cc8dbfb 
Co-authored-by: Romain B. <8530352+Rom1-B@users.noreply.github.com>
@stonebuzz @Rom1-B
Update ajax/reorder.php
5e09b44 
Co-authored-by: Romain B. <8530352+Rom1-B@users.noreply.github.com>
@stonebuzz @Rom1-B
Update inc/labeltranslation.class.php
5db9635 
Co-authored-by: Romain B. <8530352+Rom1-B@users.noreply.github.com>
@stonebuzz @Rom1-B
Update inc/profile.class.php
a72730e 
Co-authored-by: Romain B. <8530352+Rom1-B@users.noreply.github.com>
@stonebuzz @Rom1-B
Update inc/toolbox.class.php
2ee47d0 
Co-authored-by: Romain B. <8530352+Rom1-B@users.noreply.github.com>
@stonebuzz stonebuzz requested a review from Rom1-B October 10, 2025 09:49
Rom1-B
Rom1-B requested changes Oct 10, 2025
View reviewed changes
Copy link
Contributor

@Rom1-B Rom1-B left a comment

Choose a reason for hiding this comment

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

Still need to clean up all unnecessary include() statements

@stonebuzz stonebuzz requested a review from Rom1-B October 10, 2025 12:24
Rom1-B
Rom1-B reviewed Oct 10, 2025
View reviewed changes
@stonebuzz stonebuzz merged commit de37ab7 into main Oct 10, 2025
3 checks passed
@stonebuzz stonebuzz deleted the fix_fix_rector branch October 10, 2025 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Rom1-B Rom1-B Rom1-B approved these changes

Assignees

@stonebuzz stonebuzz

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stonebuzz @Rom1-B