Problem/Motivation

See parent issue for a detailed description.

Proposed resolution

Modify Gate process plugins to use InvalidArgumentException and to validate configuration in the constructor instead of ::tranform method.

Issue fork migrate_plus-3261273

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

dinazaur created an issue. See original summary.

dinazaur opened merge request !34

dinazaur’s picture

dinazaur CreditAttribution: dinazaur as a volunteer and at DevBranch, Drupal Ukraine Community commented
Assigned: dinazaur » Unassigned
Status: Active » Needs review
matroskeen’s picture

matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
CreditAttribution: matroskeen as a volunteer and at DevBranch, Evolving Web, Drupal Ukraine Community commented
Status: Needs review » Needs work

There is a merge conflict because of #3256823: Simplify code for the gate process plugin.

  • Matroskeen committed ef77138 on 8.x-5.x authored by dinazaur 
    Issue #3261273 by dinazaur, Matroskeen: Throw consistent exceptions on \...
matroskeen’s picture

matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
CreditAttribution: matroskeen as a volunteer and at DevBranch, Evolving Web, Drupal Ukraine Community commented
Status: Needs work » Fixed
Issue tags: +LutskGCW22

I went ahead and fixed the merge conflict.
Thank you!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.