Skip to content
This repository was archived by the owner on Nov 5, 2022. It is now read-only.

Update composer packages #34

Merged
merged 14 commits into from
Mar 4, 2021
Merged

Conversation

kenjis
Copy link
Member

@kenjis kenjis commented Mar 2, 2021

We can't run composer install with Composer v2.
This PR fixes it.

$ composer install
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Your lock file does not contain a compatible set of packages. Please run composer update.

  Problem 1
    - dealerdirect/phpcodesniffer-composer-installer is locked to version v0.5.0 and an update of this package was not requested.
    - dealerdirect/phpcodesniffer-composer-installer v0.5.0 requires composer-plugin-api ^1.0 -> found composer-plugin-api[2.0.0] but it does not match the constraint.

You are using Composer 2, which some of your plugins seem to be incompatible with. Make sure you update your plugins or report a plugin-issue to ask them to support Composer 2.

Includes:

  • update phpcodesniffer-composer-installer for composer v2
  • update PHPUnit to ^7.0
  • update Travis PHP versions
  • fix coding style with vendor/squizlabs/php_codesniffer/phpcs.xml.dist
  • remove composer.lock
  • add composer test command

louisl and others added 8 commits December 12, 2017 08:31
PHPUnit offers the possibility to use a `.dist` file for project configurations.
Using that, will allow individual developers to overrule the project file with their own configuration which may, for instance, contain environment variables local to their system.
…config-file

Build: rename the phpunit config file
- dep: standard autoinstaller
set project type to "phpcodesniffer-standard"
Require usage of elseif instead of else if
@kenjis
Copy link
Member Author

kenjis commented Mar 2, 2021

$ vendor/bin/phpunit --filter CodeIgniter4
PHPUnit 6.5.14 by Sebastian Bergmann and contributors.

.........                                                           9 / 9 (100%)

9 sniff test files generated 25 unique error codes; 22 were fixable (88%)

Time: 3.51 seconds, Memory: 30.00MB

OK (9 tests, 0 assertions)

Generating code coverage report in Clover XML format ... done

Generating code coverage report in HTML format ... done

@kenjis
Copy link
Member Author

kenjis commented Mar 2, 2021

It seems we must update PHPUnit to version 9.3.

https://travis-ci.org/github/codeigniter4/coding-standard/jobs/761060422

Your requirements could not be resolved to an installable set of packages.

  Problem 1

    - phpunit/phpunit[6.5.0, ..., 6.5.14] require php ^7.0 -> your php version (8.0.2) does not satisfy that requirement.

    - Root composer.json requires phpunit/phpunit ^6.5 -> satisfiable by phpunit/phpunit[6.5.0, ..., 6.5.14].

@kenjis kenjis marked this pull request as draft March 2, 2021 03:48
@kenjis
Copy link
Member Author

kenjis commented Mar 2, 2021

@kenjis kenjis changed the title Update phpcodesniffer-composer-installer version Update composer packages Mar 2, 2021
@kenjis kenjis marked this pull request as ready for review March 2, 2021 04:48
@kenjis
Copy link
Member Author

kenjis commented Mar 2, 2021

@louisl Could you review this?

@louisl
Copy link
Collaborator

louisl commented Mar 2, 2021

@kenjis Thanks for the PR, I'll check it out ASAP.

@louisl louisl changed the base branch from master to develop March 4, 2021 10:01
@louisl louisl merged commit 113687c into codeigniter4:develop Mar 4, 2021
@kenjis kenjis deleted the update-composer.json branch March 4, 2021 22:17
@kenjis
Copy link
Member Author

kenjis commented Mar 4, 2021

@louisl Thanks!

@kenjis
Copy link
Member Author

kenjis commented Mar 4, 2021

@louisl When I composer update in CI4, codeigniter4-standard does not update.
See #37

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants