Standardize SQL Conventions

Issue #594 resolved
Ghislain Hachey created an issue

Minor details. But brackets surrounding objects should be consistent across SQLDeltas. See figure below. I think they should all contain the bracket or all not contain the bracket. In other words, I would not want to have to specify a SQLDelta config to ignore brackets.

Comments (3)

  1. Brian Lewis repo owner

    The DbScripter program used to generate the scripts for inclusion in the repo does some “lint-checking” on various issues, including adding brackets around object names.

    The main point being, we don;t want “false positives” changes in the repo due to these issues. DbScripter also removes trailing spaces.

    The one place where you see this in delta is in trigger definitions, where it seems to be not always consistent - if you look at the source script, you may well find that the brackets are actually there.

    But you are right the intention is to have brackets everywhere that a name is defined : is in CREATE ….. <object> and also in TRIGGER ON

    Set Ignore Square Brackets = No in the Sql Delta config.

  2. Log in to comment