Skip to content

Conversation

@kenjis
Copy link
Contributor

@kenjis kenjis commented Sep 11, 2023

See codeigniter4/CodeIgniter4#7819

The following is not updated:

  • phpunit.yml
  • rector.yml

@kenjis kenjis marked this pull request as ready for review September 11, 2023 11:49
@Arhell Arhell requested a review from MGatner September 12, 2023 00:07
@MGatner
Copy link
Collaborator

MGatner commented Sep 12, 2023

Why not rector.yml? Some dependency issues as Settings?

@kenjis
Copy link
Contributor Author

kenjis commented Sep 13, 2023

Rector is not required in this repository.

website/composer.json

Lines 34 to 37 in ed010b2

"require-dev": {
"codeigniter4/devkit": "^1.0",
"tatter/patches": "^2.0"
},

@MGatner
Copy link
Collaborator

MGatner commented Sep 13, 2023

We should probably change that to use our standard. The YAML is installing Rector globally but the version is way behind: ^0.15.1

Doesn't have to be part of this PR but I would like to see this consistent with our other repos.

@kenjis kenjis merged commit bcbd7d6 into codeigniter4projects:develop Sep 14, 2023
@kenjis kenjis deleted the reuse-ga-workflows branch September 14, 2023 00:12
@kenjis
Copy link
Contributor Author

kenjis commented Sep 14, 2023

Let's update it after the new devkit release.

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.

3 participants