-
-
Notifications
You must be signed in to change notification settings - Fork 275
Ckeditor5 with view source button #1089
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
base: release-3.7.0
Are you sure you want to change the base?
Conversation
.github/workflows/upgrade.yml
Outdated
- name: Report Versions and show ENV vars | ||
run: | | ||
google-chrome --version | ||
lsb_release -a | ||
php -v | ||
chromedriver -v | ||
geckodriver -V | ||
which geckodriver | ||
which chromedriver | ||
firefox -v | ||
vendor/bin/behat -V | ||
mysql --version | ||
mysqladmin --version | ||
export | ||
|
||
- name: Create Database | ||
run: | | ||
sudo systemctl start mysql.service | ||
sudo mysql -proot -e 'drop database if exists phplistdb; create database phplistdb;' | ||
sudo mysql -proot -e 'create user phplist@"%" identified by "phplist"; grant all on phplistdb.* to phplist@"%"' | ||
sudo gunzip -c tests/phplist-3.6.sql.gz | sudo mysql -proot phplistdb | ||
|
||
- name: Set bootlist theme | ||
run: | | ||
cd $GITHUB_WORKSPACE/public_html/lists/admin/ui/ | ||
wget https://github.com/phpList/phplist-ui-bootlist/archive/master.tar.gz | ||
tar -xzf master.tar.gz | ||
mv phplist-ui-bootlist-master phplist-ui-bootlist | ||
rm master.tar.gz | ||
|
||
- name: Start Test Server | ||
run: | | ||
cd $GITHUB_WORKSPACE | ||
cp -fv tests/ci/behat.yml tests/behat.yml | ||
cp -fv tests/ci/config.php public_html/lists/config/config.php | ||
mkdir -p output/screenshots | ||
touch output/screenshots/README.md | ||
mkdir -p build/mails | ||
smtp-sink -c -d "output/mails/%Y%m%d%H/%M." 2500 1024 & | ||
./bin/start-selenium > output/selenium.log 2>&1 & | ||
sleep 5 | ||
sudo php -S 0.0.0.0:80 -t public_html > /dev/null 2>&1 & | ||
|
||
- name: Check PHP syntax errors | ||
uses: overtrue/[email protected] | ||
with: | ||
path: ./public_html | ||
|
||
- name: Upgrade from 3.6 to 3.7 | ||
run: | | ||
php public_html/lists/admin/index.php -c $GITHUB_WORKSPACE/public_html/lists/config/config.php -p upgrade | ||
|
||
- name: Run BDD Tests UI | ||
run: | | ||
cd $GITHUB_WORKSPACE/tests | ||
../vendor/bin/behat -p chrome -f progress --tags="~@initialise && ~@wip" | ||
|
||
- name: Reset Database | ||
run: | | ||
sudo mysql -proot -e 'drop database if exists phplistdb; create database phplistdb;' | ||
|
||
- name: Run BDD Tests UI | ||
run: | | ||
cd $GITHUB_WORKSPACE/tests | ||
../vendor/bin/behat -p chrome -f progress --stop-on-failure --tags=@initialise | ||
../vendor/bin/behat -p chrome -f progress --tags="~@initialise && ~@wip" | ||
|
||
- name: Run BDD Tests CLI | ||
run: | | ||
cd $GITHUB_WORKSPACE | ||
export [email protected] | ||
export ADMIN_PASSWORD=Mypassword123+ | ||
export ORGANISATION_NAME="phpList" | ||
export ADMIN_NAME="phpList Administrator" | ||
php public_html/lists/admin/index.php -c $GITHUB_WORKSPACE/public_html/lists/config/config.php -p initialise -f | ||
cd $GITHUB_WORKSPACE/tests | ||
../vendor/bin/behat -p chrome -f progress --tags="~@initialise && ~@wip" | ||
|
||
- name: copy screenshots | ||
if: always() | ||
run: | | ||
cd $GITHUB_WORKSPACE | ||
images=$(ls -l /tmp/*.png 2>/dev/null | wc -l) | ||
[[ $images -gt 0 ]] && cp -v /tmp/*.png output/screenshots | ||
exit 0; | ||
|
||
- name: Upload the screenshots | ||
if: always() | ||
uses: actions/upload-artifact@v4 | ||
with: | ||
path: "output" | ||
name: "behat output ${{ matrix.php-version }}" | ||
retention-days: 3 | ||
|
||
- name: Display output | ||
run: | | ||
cd $GITHUB_WORKSPACE | ||
# find . -type f | ||
# cat output/selenium.log | ||
if: ${{ failure() }} |
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 19 days ago
To fix the problem, add a permissions
block at the root of the workflow. This block will explicitly define the minimal permissions required for the workflow to run securely. Based on the workflow's operations (e.g., using actions like checkout
and upload-artifact
), the minimal permissions required are likely contents: read
and potentially contents: write
or other resource-specific permissions if needed.
- Add the
permissions
key at the root level of the workflow file. - Define the least privileges required for the workflow's tasks. For example:
contents: read
to allow reading repository content.contents: write
if the workflow modifies the repository (e.g., pushes changes).- Additional permissions if specific actions (e.g., handling pull requests or issues) require them.
-
Copy modified lines R3-R4
@@ -1,5 +1,7 @@ | ||
|
||
name: Upgrade | ||
permissions: | ||
contents: read | ||
|
||
on: | ||
workflow_dispatch: |
Description
Update Ckeditor5 to version with view source button
Contributor License Agreement
before we can accept your PR, if you haven't done this yet, please sign the
Contributor License Agreement at https://www.phplist.com/cla
Many thanks
the phpList Team
-->
Related Issue
TatevikGr/phplist-plugin-ckeditor5#1
Screenshots (if appropriate):