-
Notifications
You must be signed in to change notification settings - Fork 35
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.
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
Feature/admin pages menu #1810
base: master
Are you sure you want to change the base?
Feature/admin pages menu #1810
Changes from all commits
283d3e7
0be3152
275f604
cea1e3b
2ea1eb7
7774e88
bbaeea1
8d5a825
d389e5b
68a6f64
e27d522
019a12b
03a576f
a2b02d5
4dd60f8
9495699
6d4b82b
4a80973
c9ffd06
00a0ea0
49620c7
6a2a670
9d4cdc1
834d74d
05ff1f9
193adc9
9d2ae53
5b571ac
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -42,8 +42,8 @@ env: | |
- WP_ADMIN_PASSWORD="password" | ||
|
||
before_install: | ||
# Rollback Composer to version 1. | ||
- composer self-update --rollback | ||
# Set Composer to version 2. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This looks like a good chance to move Common build away from Travis CI, as is the case for the other plugins, and just remove any configuration file related to it. |
||
- composer self-update --2 | ||
|
||
# create the databases needed for the tests | ||
- mysql -e "create database IF NOT EXISTS $DB_NAME;" -uroot | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,31 +9,37 @@ | |
"homepage": "https://theeventscalendar.com", | ||
"license": "GPL-2.0", | ||
"require-dev": { | ||
"bordoni/phpass": "0.3.5", | ||
"automattic/vipwpcs": "^2.0", | ||
"bordoni/phpass": "0.3.5", | ||
"brianhenryie/strauss": "^0.11.0", | ||
"codeception/codeception": "^2.5.5", | ||
"dealerdirect/phpcodesniffer-composer-installer": "^0.7.2", | ||
"lucatume/codeception-snapshot-assertions": "^0.2.4", | ||
"lucatume/function-mocker-le": "^1.0", | ||
"lucatume/wp-browser": "^3.0.14", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Update this to |
||
"phpunit/phpunit": "~6.0", | ||
"the-events-calendar/coding-standards": "dev-master", | ||
"the-events-calendar/tec-testing-facilities": "dev-master", | ||
"phpunit/phpunit": "~6.0", | ||
"wp-cli/checksum-command": "1.0.5", | ||
"wp-coding-standards/wpcs": "^2.1", | ||
"lucatume/codeception-snapshot-assertions": "^0.2.4" | ||
"wp-coding-standards/wpcs": "^2.1" | ||
}, | ||
"minimum-stability": "dev", | ||
"prefer-stable": true, | ||
"config": { | ||
"preferred-install": "dist", | ||
"platform": { | ||
"php": "7.3.33" | ||
"php": "7.4.0" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Minimum required version of PHP in the plugins is 7.3; the |
||
}, | ||
"allow-plugins": { | ||
"kylekatarnls/update-helper": true, | ||
"dealerdirect/phpcodesniffer-composer-installer": true | ||
} | ||
}, | ||
"require": { | ||
"lucatume/di52": "~2.1.5", | ||
"firebase/php-jwt": "~5.0.0", | ||
"monolog/monolog": "1.24.*" | ||
"monolog/monolog": "1.24.*", | ||
"iconicwp/wordpress-settings-framework": "dev-master" | ||
}, | ||
"repositories": [ | ||
{ | ||
|
@@ -47,13 +53,24 @@ | |
"type": "github", | ||
"url": "https://github.com/the-events-calendar/coding-standards", | ||
"no-api": true | ||
}, | ||
{ | ||
"name": "iconicwp/wordpress-settings-framework", | ||
"type": "github", | ||
"url": "https://github.com/iconicwp/WordPress-Settings-Framework", | ||
"no-api": true | ||
} | ||
], | ||
"scripts": { | ||
"strauss": [ | ||
"vendor/bin/strauss" | ||
], | ||
"post-install-cmd": [ | ||
"@strauss", | ||
"xrstf\\Composer52\\Generator::onPostInstallCmd" | ||
], | ||
"post-update-cmd": [ | ||
"@strauss", | ||
"xrstf\\Composer52\\Generator::onPostInstallCmd" | ||
], | ||
"post-autoload-dump": [ | ||
|
@@ -65,5 +82,21 @@ | |
"Tribe\\": "src/Tribe/", | ||
"TEC\\Common\\": "src/Common/" | ||
} | ||
}, | ||
"extra": { | ||
"strauss": { | ||
"target_directory": "vendor/strauss", | ||
"namespace_prefix": "TEC\\Common\\", | ||
"classmap_prefix": "TEC_", | ||
"constant_prefix": "TEC_", | ||
"packages": [ | ||
"vendor/iconicwp/wordpress-settings-framework" | ||
], | ||
"exclude_from_prefix": { | ||
"packages": [ | ||
"dealerdirect/phpcodesniffer-composer-installer" | ||
] | ||
} | ||
} | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know we used to ignore the
tric
configuration files in Common, but I think we should not: TEC and other plugins are includingtric
configuration files in VCS.