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.

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

Dependency updates #80

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Dependency updates #80

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
f803a09
Update deployer to newer version
Dec 17, 2020
d0af169
Update travis instances.
Dec 17, 2020
2293a42
Fixing Travis build
Dec 17, 2020
7345f69
Fixing Travis build
Dec 17, 2020
c2b5783
Fixing Travis build
Dec 17, 2020
7683cb3
Fixing Travis build
Dec 17, 2020
008bf6c
Fixing Travis build
Dec 17, 2020
6331c28
Fixing Travis build
Dec 17, 2020
7104838
Fixing Travis build
Dec 17, 2020
276121b
Fixing Travis build
Dec 17, 2020
f098d6f
Fixing Travis build
Dec 17, 2020
b89fa51
Reverted back to using system composer version (Composer 2 works)
erfanimani Oct 2, 2022
6ed9295
Updated dependencies (deployer, grumphp and phpstan)
erfanimani Oct 2, 2022
908485e
Added back composer install
erfanimani Oct 2, 2022
e0691c4
Removed PHP 7.2 and 7.3 from CI tests
erfanimani Oct 3, 2022
4ff0ef4
Stopped supporting PHP 7.2 and 7.3
erfanimani Oct 3, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions .travis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
language: php

php:
- 7.0
- 7.1
- 7.2
- 7.4
- 8.1

before_install:
- sudo apt-get -qq update
Expand Down
12 changes: 9 additions & 3 deletions composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,19 +12,25 @@
}
],
"require": {
"deployer/dist": "^6.0"
"php": "~7.4.0||~8.1.0",
"deployer/deployer": "^7.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

😲 That works without updating any of the current tasks?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd say at least the sample deploy.php files should be updated, because according to https://deployer.org/docs/7.x/UPGRADE, the hosts definition changed quite a bit?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oops, yes, this PR isn't ready yet.

},
"require-dev": {
"phpro/grumphp": "^0.14.0",
"phpro/grumphp": "^0.18.0||^1.13",
"phpmd/phpmd": "^2.6",
"nikic/php-parser": "^3.1",
"squizlabs/php_codesniffer": "^3.2",
"phpstan/phpstan": "^0.9.2",
"phpstan/phpstan": "^0.9.2||^1.8",
"friendsofphp/php-cs-fixer": "^2.11"
},
"autoload": {
"files": [
"autoload.php"
]
},
"config": {
"allow-plugins": {
"phpro/grumphp": true
}
}
}
19 changes: 1 addition & 18 deletions grumphp.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,26 +1,9 @@
parameters:
bin_dir: ./vendor/bin
git_dir: .
grumphp:
stop_on_failure: false
hide_circumvention_tip: true
ascii:
failed: ~
succeeded: ~
composer:
file: ./composer.json
no_check_all: true
no_check_lock: false
no_check_publish: false
with_dependencies: false
strict: false
git_blacklist:
keywords:
- "die("
- "var_dump("
- "exit;"
- "console.log("
triggered_by: ['php']
regexp_type: G
tasks:
phpmd:
ruleset: ['phpmd.xml']
Expand Down