Problem/Motivation

It is a good practice to use readonly properties for injected dependencies.

Issue fork graphql_webform-3475289

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

pfrenssen created an issue. See original summary.

pfrenssen opened merge request !43

pfrenssen’s picture

pfrenssen
Location Sofia
commented

  • pfrenssen committed 7e2a743c on 2.x 
    Issue #3475289 by pfrenssen: Use readonly properties where possible
pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Active » Fixed
hazn’s picture

hazn commented
Status: Fixed » Reviewed & tested by the community

LGTM :)

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Reviewed & tested by the community » Fixed

Thanks for the review! This has already merged, marking as fixed.

Status: Fixed » Closed (fixed)

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