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.

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

Sync CI pipeline with moodlehq/moodle-plugin-ci #22

Merged
merged 1 commit into from
Mar 31, 2025

Conversation

erseco
Copy link
Collaborator

@erseco erseco commented Mar 30, 2025

This PR updates .github/workflows/moodle-ci.yml to match the latest version from moodlehq/moodle-plugin-ci.
Includes updated PHP/PostgreSQL versions and improved Behat config.

@erseco erseco requested a review from Copilot March 30, 2025 08:21
@erseco erseco self-assigned this Mar 30, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR syncs the CI pipeline configuration with the latest version from moodlehq/moodle-plugin-ci to update PHP/PostgreSQL versions and enhance Behat testing.

  • Upgrade PostgreSQL from version 13 to 14.
  • Update the PHP testing matrix and reduce the supported Moodle branches.
  • Enhance the Behat command with an additional flag.
Comments suppressed due to low confidence (3)

.github/workflows/moodle-ci.yml:33

  • Verify that dropping PHP 8.3 support was intentional and that all tests are compatible with the revised PHP versions.
php: ['7.4', '8.0', '8.1']

.github/workflows/moodle-ci.yml:34

  • Confirm that the removal of 'MOODLE_404_STABLE' is deliberate, and update relevant documentation if needed.
moodle-branch: ['MOODLE_401_STABLE']

.github/workflows/moodle-ci.yml:109

  • Ensure that adding the '--scss-deprecations' flag aligns with the expected behavior in your Behat tests and does not introduce side effects.
run: moodle-plugin-ci behat --profile chrome --scss-deprecations

@ignaciogros ignaciogros merged commit f2ce08c into main Mar 31, 2025
0 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants