Skip to content

Conversation

HugoKornelis
Copy link
Contributor

@PRMerger17
Copy link
Contributor

@HugoKornelis : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@PRMerger19
Copy link
Contributor

@VanMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@VanMSFT
Copy link
Member

VanMSFT commented Oct 21, 2020

Hey @HugoKornelis - Thanks for this, but column_position and [ ,...n ] should not be removed. It is part of the syntax. https://docs.microsoft.com/sql/t-sql/language-elements/transact-sql-syntax-conventions-transact-sql. I think carets make more sense in this case.

@HugoKornelis
Copy link
Contributor Author

HugoKornelis commented Oct 21, 2020 via email

@PRMerger9
Copy link
Contributor

@VanMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@VanMSFT
Copy link
Member

VanMSFT commented Oct 21, 2020

Thanks, @HugoKornelis! I see your point. I'll go ahead and approve the changes. Thanks again for taking the time on this.

#sign-off

@ktoliver ktoliver merged commit 57c4cd9 into MicrosoftDocs:live Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants