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

refactor: enable Rector Code Quality level, apply level 9 for SimplifyConditionsRector #9227

Conversation

samsonasik
Copy link
Member

Description

We can enable codeQuality sets once we are on max level. This PR apply level one by one, and I can see it apply on level 9, with cleared rules manual definitions that included in previous level.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@samsonasik samsonasik changed the title Refactor: enable Rector Code Quality level, apply level 9 for SimplifyConditionsRector refactor: enable Rector Code Quality level, apply level 9 for SimplifyConditionsRector Oct 15, 2024
@samsonasik
Copy link
Member Author

Ready to merge 👍

@samsonasik
Copy link
Member Author

Thank you @kenjis @paulbalandan for the review, let's merge ;)

@samsonasik samsonasik merged commit c7ab0d1 into codeigniter4:develop Oct 19, 2024
39 checks passed
@samsonasik samsonasik deleted the refactor-enable-rector-code-quality-level branch October 19, 2024 03:56
@kenjis kenjis added the testing Pull requests that changes tests only label Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Pull requests that changes tests only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants