Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Move enlightn/security-checker to "suggest" #5078

Merged
merged 18 commits into from
Feb 21, 2022
Merged

Move enlightn/security-checker to "suggest" #5078

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
b769fde
Move enlightn/security-checker to "suggest"
greg-1-anderson Feb 18, 2022
629e714
Update to grasmash/expander 2.0.1
greg-1-anderson Feb 19, 2022
620cdb3
Prepare for league/container ^4
greg-1-anderson Feb 19, 2022
d1a6094
Can't have more than one test class named CoreTest
greg-1-anderson Feb 19, 2022
0931c51
Update dependencies
greg-1-anderson Feb 19, 2022
a6ffa5d
Maybe this might fix Drupal 10, maybe it might break other versions.
greg-1-anderson Feb 19, 2022
a18707a
Code style
greg-1-anderson Feb 19, 2022
688a114
Rename test class
greg-1-anderson Feb 19, 2022
9f57b35
The function "module_load_install()" was marked as deprecated in Drup…
lhridley Feb 13, 2022
037728b
Install enlightn/security-checker for Windows tests. Skip security ch…
greg-1-anderson Feb 20, 2022
87bff30
Stop using deprecated drupal_get_installed_schema_version when new re…
greg-1-anderson Feb 20, 2022
17375da
Put update script name into a method
greg-1-anderson Feb 20, 2022
c2509f9
Avoid using drupal_set_installed_schema_version, removed in Drupal 10
greg-1-anderson Feb 20, 2022
75cb6ee
Code style
greg-1-anderson Feb 20, 2022
a3017d1
Fix update to requirements check implementation.
greg-1-anderson Feb 20, 2022
e8da222
Code style
greg-1-anderson Feb 20, 2022
056402b
Update dependencies and remove an optional backslash
greg-1-anderson Feb 20, 2022
04b5ac0
Correctly skip security tests on Drupal 10
greg-1-anderson Feb 20, 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
4 changes: 3 additions & 1 deletion .circleci/config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ test_74_steps: &test74steps
steps:
- checkout
- run: cp .docker/zz-php.ini /usr/local/etc/php/conf.d/
- run: composer -n install
- run: composer -n require enlightn/security-checker --no-update
- run: composer -n update
- run: mkdir -p /tmp/results
- run: composer -n lint
- run: composer -n unit -- --log-junit /tmp/results/unit.junit.xml
Expand Down Expand Up @@ -103,6 +104,7 @@ jobs:
- run: cp .docker/zz-php.ini /usr/local/etc/php/conf.d/
- run: composer -n require --dev drupal/core-recommended:9.2.8 --no-update
- run: composer -n require symfony/polyfill-php80:"1.23 as 1.20" --no-update
- run: composer -n require enlightn/security-checker --no-update
- run: php --version
- run: composer -n update
- run: composer -n phpunit -- --testsuite integration --filter=testInsecureDrupalPackage --stop-on-skipped
Expand Down
1 change: 1 addition & 0 deletions appveyor.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ install:
- php -r "readfile('http://getcomposer.org/installer');" | php
#Install dependencies via Composer
- php composer.phar install --prefer-dist -n
- php composer.phar require enlightn/security-checker -n
- SET PATH=%APPVEYOR_BUILD_FOLDER%;%APPVEYOR_BUILD_FOLDER%/vendor/bin;%PATH%
# Uncomment this and on_finish line below to enable RDP into build machine https://www.appveyor.com/docs/how-to/rdp-to-build-worker/
# - ps: iex ((new-object net.webclient).DownloadString('https://raw.githubusercontent.com/appveyor/ci/master/scripts/enable-rdp.ps1'))
Expand Down
10 changes: 6 additions & 4 deletions composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,12 +37,11 @@
"consolidation/annotated-command": "^4.5",
"consolidation/config": "^2",
"consolidation/filter-via-dot-access-data": "^2",
"consolidation/robo": "^3",
"consolidation/robo": "^3.0.9",
"consolidation/site-alias": "^3.1.3",
"consolidation/site-process": "^4.1.3",
"enlightn/security-checker": "^1",
"consolidation/site-process": "^4.1.3 || ^5",
"guzzlehttp/guzzle": "^6.3 || ^7.0",
"league/container": "^3.4",
"league/container": "^3.4 || ^4",
"psr/log": "~1.0",
"psy/psysh": "~0.11",
"symfony/event-dispatcher": "^4.0 || ^5.0 || ^6.0",
Expand All @@ -65,6 +64,9 @@
"vlucas/phpdotenv": "^2.4",
"yoast/phpunit-polyfills": "^0.2.0"
},
"suggest": {
"enlightn/security-checker": "Required to use pm:security command"
},
"conflict": {
"drupal/core": "< 9.2",
"drupal/migrate_run": "*",
Expand Down
Loading