Skip to main content

Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

    •  
      request #13361 Require the nullability symbol to be present when the default value is null
    Actions
    Infos
    #13361
    Thomas Gerbet (tgerbet)
    2019-05-09 17:03
    2019-05-03 16:39
    14482
    Details
    Require the nullability symbol to be present when the default value is null
    It is more explicit and this discussion [0] from two years ago from the doctrine/repo also points out it can hide a LSP violation.

    Now seems to be a good time to introduce this rule as we start to aggressively type scalar values.



    [0] https://github.com/doctrine/common/pull/805#discussion_r128762923
    Dev tools
    Empty
    Empty
    • [ ] enhancement
    • [x] internal improvement
    Empty
    Stage
    Empty
    Closed
    2019-05-09
    Attachments
    Empty
    References
    Referencing request #13361

    Git commit

    tuleap/tuleap/stable

    Merge commit 'refs/changes/02/14902/1' of ssh://gerrit.tuleap.net:29418/tuleap into HEAD 7196a84b21
    User avatar Nicolas Terray (nterray) , 2019-05-06 10:03
    request #13361: Require the nullability symbol to be present when the default value is null 1d08153d44
    User avatar Thomas Gerbet (tgerbet) , 2019-05-03 17:03
    request #13361: Require the nullability symbol to be present when the default value is null df6fbaf6cb
    User avatar Thomas Gerbet (tgerbet) , 2019-05-07 11:31
    Referenced by request #13361

    Artifact Tracker v5

    rel #13072 11.2

    Other

    gerrit #14902
    gerrit #14920
    Display settings

    Follow-ups