Skip to content
Snippets Groups Projects

issue fixed

All threads resolved!

phpcs issue fixed #3398496

Merge request reports

Approved by
Code Quality is loading
Test summary results are being parsed

Merged by Yurii PanchukYurii Panchuk 1 year ago (Nov 27, 2023 7:46pm UTC)

Merge details

  • Changes merged into 8.x-1.x with 75a5257f (commits were squashed).
  • Did not delete the source branch.

Pipeline #55821 passed with warnings

Pipeline passed with warnings for 75a5257f on 8.x-1.x

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Yurii Panchuk
  • Yurii Panchuk
  • Yurii Panchuk
  • Yurii Panchuk
  • @chetan-11 thanks!

    The code looks much better. Tests are green, but there a few code styling problems found in MR.

    Also, you have changed some variable's names but forgot to update them in the code.

  • chetan added 1 commit

    added 1 commit

    Compare with previous version

  • Yurii Panchuk added 4 commits

    added 4 commits

    Compare with previous version

  • Yurii Panchuk added 2 commits

    added 2 commits

    Compare with previous version

  • Yurii Panchuk resolved all threads

    resolved all threads

  • Yurii Panchuk added 1 commit

    added 1 commit

    Compare with previous version

  • Yurii Panchuk added 1 commit

    added 1 commit

    Compare with previous version

  • Yurii Panchuk approved this merge request

    approved this merge request

  • merged

  • Please register or sign in to reply
    Loading