Skip to content

Disable xdebug in integration tests #23

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

Closed
wants to merge 1 commit into from

Conversation

ihor-sviziev
Copy link
Collaborator

@ihor-sviziev ihor-sviziev commented Aug 5, 2022

  1. disable xdebug as it's not needed in integration tests
  2. install composer via shivammathur/setup-php github action
  3. update actions/checkout to v3

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Fixes: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

damienwebdev added a commit that referenced this pull request Aug 5, 2022
Co-authored-by: Ihor Sviziev <ihor-sviziev@users.noreply.github.com>
damienwebdev added a commit that referenced this pull request Aug 5, 2022
I also disabled Xdebug because it is not used in integration tests.

Co-authored-by: Ihor Sviziev <ihor-sviziev@users.noreply.github.com>
damienwebdev added a commit that referenced this pull request Aug 5, 2022
Co-authored-by: Ihor Sviziev <ihor-sviziev@users.noreply.github.com>
damienwebdev added a commit that referenced this pull request Aug 5, 2022
Co-authored-by: Ihor Sviziev <ihor-sviziev@users.noreply.github.com>
damienwebdev added a commit that referenced this pull request Aug 5, 2022
I also disabled Xdebug because it is not used in integration tests.

Co-authored-by: Ihor Sviziev <ihor-sviziev@users.noreply.github.com>
damienwebdev added a commit that referenced this pull request Aug 5, 2022
Co-authored-by: Ihor Sviziev <ihor-sviziev@users.noreply.github.com>
@damienwebdev
Copy link
Member

Merged as

86686e6
49bbcc8
8a209b9

@ihor-sviziev ihor-sviziev deleted the patch-2 branch August 5, 2022 13:16
damienwebdev pushed a commit that referenced this pull request Mar 25, 2025
@gray-bot gray-bot mentioned this pull request Mar 25, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants