Skip to content

Commit

Permalink
Merge pull request #416 from ergebnis/feature/composer-normalize
Browse files Browse the repository at this point in the history
Enhancement: Require `ergebnis/composer-normalize`
  • Loading branch information
localheinz authored Jul 16, 2024
2 parents 54fb6d8 + efa0ad1 commit a330427
Show file tree
Hide file tree
Showing 5 changed files with 419 additions and 9 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/integrate.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@ on: # yamllint disable-line rule:truthy
branches:
- "main"

env:
COMPOSER_ROOT_VERSION: "3.4-dev"

jobs:
code-coverage:
name: "Code Coverage"
Expand Down Expand Up @@ -128,10 +131,10 @@ jobs:
- name: "Install dependencies with phive"
uses: "ergebnis/.github/actions/phive/[email protected]"
with:
trust-gpg-keys: "0xC00543248C87FB13,0x033E5F8D801A2F8D"
trust-gpg-keys: "0x033E5F8D801A2F8D"

- name: "Run ergebnis/composer-normalize"
run: ".phive/composer-normalize --ansi --dry-run"
run: "composer normalize --ansi --dry-run"

- name: "Cache cache directory for friendsofphp/php-cs-fixer"
uses: "actions/[email protected]"
Expand Down Expand Up @@ -196,7 +199,7 @@ jobs:
- name: "Install dependencies with phive"
uses: "ergebnis/.github/actions/phive/[email protected]"
with:
trust-gpg-keys: "0xC00543248C87FB13,0x033E5F8D801A2F8D"
trust-gpg-keys: "0x033E5F8D801A2F8D"

- name: "Run maglnet/composer-require-checker"
run: ".phive/composer-require-checker check --ansi --config-file=$(pwd)/composer-require-checker.json --verbose"
Expand Down
1 change: 0 additions & 1 deletion .phive/phars.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
<?xml version="1.0" encoding="UTF-8"?>
<phive xmlns="https://phar.io/phive">
<phar name="composer-normalize" version="^2.42.0" installed="2.42.0" location="./.phive/composer-normalize" copy="false"/>
<phar name="composer-require-checker" version="^3.8.0" installed="3.8.0" location="./.phive/composer-require-checker" copy="false"/>
</phive>
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ code-coverage: vendor ## Collects coverage from running unit tests with phpunit/
.PHONY: coding-standards
coding-standards: phive vendor ## Lints YAML files with yamllint, normalizes composer.json with ergebnis/composer-normalize, and fixes code style issues with friendsofphp/php-cs-fixer
yamllint -c .yamllint.yaml --strict .
.phive/composer-normalize
composer normalize
vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.php --diff --show-progress=dots --verbose

.PHONY: dependency-analysis
Expand All @@ -25,7 +25,7 @@ mutation-tests: vendor ## Runs mutation tests with infection/infection

.PHONY: phive
phive: .phive ## Installs dependencies with phive
PHIVE_HOME=.build/phive phive install --trust-gpg-keys 0xC00543248C87FB13,0x033E5F8D801A2F8D
PHIVE_HOME=.build/phive phive install --trust-gpg-keys 0x033E5F8D801A2F8D

.PHONY: refactoring
refactoring: vendor ## Runs automated refactoring with rector/rector
Expand Down
5 changes: 5 additions & 0 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
"php": "~7.4.0 || ~8.0.0 || ~8.1.0 || ~8.2.0 || ~8.3.0"
},
"require-dev": {
"ergebnis/composer-normalize": "^2.43.0",
"ergebnis/data-provider": "^3.2.0",
"ergebnis/license": "^2.4.0",
"ergebnis/php-cs-fixer-config": "^6.32.0",
Expand All @@ -49,6 +50,7 @@
"config": {
"allow-plugins": {
"composer/package-versions-deprecated": true,
"ergebnis/composer-normalize": true,
"infection/extension-installer": true
},
"audit": {
Expand All @@ -61,6 +63,9 @@
"sort-packages": true
},
"extra": {
"branch-alias": {
"dev-main": "3.4-dev"
},
"composer-normalize": {
"indent-size": 2,
"indent-style": "space"
Expand Down
Loading

0 comments on commit a330427

Please sign in to comment.