Skip to content

Run EPV through the organization's strict EPV workflow #3

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

Merged
bonelifer merged 2 commits into from
Oct 19, 2026
Merged

Run EPV through the organization's strict EPV workflow #3

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
502ef9e
Run EPV through the organization's strict EPV workflow
bonelifer Oct 19, 2026
b327c3b
Merge branch 'master' into strict-epv
bonelifer Oct 19, 2026
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
9 changes: 9 additions & 0 deletions .github/epv-allowed.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
# EPV findings this extension keeps on purpose, checked by the epv job in
# .github/workflows/tests.yml. One finding per line as EPV prints it, without
# "on line N", with paths relative to the extension; give the reason above it.

# Released migration: it only changes the type of the user_justification column,
# which m1_initial_schema's revert_schema() drops when the extension's data is
# deleted, so there is nothing for this migration to revert. Released migrations
# must not be changed.
Error: Migration file /migrations/m2_update_schema.php is missing the revert_schema() method.
20 changes: 8 additions & 12 deletions .github/workflows/tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ env:
EXTNAME: rmcgirr83/activationjustification # Your extension vendor/package name
SNIFF: 1 # Run code sniffer on your code? 1 or 0
IMAGE_ICC: 1 # Run icc profile sniffer on your images? 1 or 0
EPV: 1 # Run EPV (Extension Pre Validator) on your code? 1 or 0
EXECUTABLE_FILES: 1 # Run check for executable files? 1 or 0
PHPBB_BRANCH: 3.3.x # The phpBB branch to run tests on

Expand All @@ -19,7 +18,7 @@ on:
- develop

jobs:
# START Basic Checks Job (EPV, code sniffer, images check, etc.)
# START Basic Checks Job (code sniffer, images check, etc.)
basic-checks:
runs-on: ubuntu-24.04
strategy:
Expand Down Expand Up @@ -59,11 +58,6 @@ jobs:
run: .github/setup-phpbb.sh $DB $PHP_VERSION $NOTESTS
working-directory: ./phpBB3

- name: Setup EPV
if: ${{ env.EPV != 0 }}
run: composer require phpbb/epv:dev-master --dev --no-interaction --ignore-platform-reqs
working-directory: ./phpBB3/phpBB

- name: Run code sniffer
if: ${{ env.SNIFF != 0 }}
env:
Expand All @@ -80,9 +74,11 @@ jobs:
if: ${{ env.EXECUTABLE_FILES != 0 }}
run: .github/ext-check-executable-files.sh ./ $EXTNAME
working-directory: ./phpBB3

- name: Run EPV
if: ${{ env.EPV != 0 }}
run: phpBB/vendor/bin/EPV.php run --dir="phpBB/ext/$EXTNAME/"
working-directory: ./phpBB3
# END Basic Checks Job

# EPV (Extension Pre Validator), failing on any Fatal or Error finding not
# listed in .github/epv-allowed.txt
epv:
uses: phpbbmodders/.github/.github/workflows/epv.yml@v1
with:
EXTNAME: rmcgirr83/activationjustification
Loading