Re: [sarif-comment] Non-material GitHub issues to fix

[David Keaton <dmk@dmk.com>]

     This comment was from a TC member, so it was moved to the TC 
mailing list.

					David

On 2020-02-17 11:34, Larry Golding (Myriad Consulting Inc) wrote:
> Please fix the following non-material issues that have been captured as 
> GitHub issues in https://github.com/oasis-tcs/sarif-spec 
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235478907&sdata=7YUNuGqFoiFsFv4hfJfkI9YasRE7VaRwq9JttUXfLXY%3D&reserved=0>:
>
>   * #422
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F422&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235488901&sdata=3wSUWoSK6xUUv%2BIgfk8jsDJC2qbDUjV5NvsivBkhufU%3D&reserved=0>:
>     Ensure all GUIDs in format examples are valid but distinguishable
>   * #430
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F430&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235498896&sdata=3u1pqwkkDq5MDbmU8pMTZFm7psrvPuehl1Hc2CG7EZc%3D&reserved=0>:
>     Conformance Clause numbering skips clause 4
>   * #431
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F431&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235498896&sdata=ZEu3DKFWgTTqR8z78Oh9LTqvztkce4118FKLsY2waO8%3D&reserved=0>:
>     Add xref 3.7.4 to graphTraversal.runGraphIndex
>   * #432
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F432&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235508892&sdata=cOqXCmm6gvTRMucUdlJe1frOKLtzjO8JgHorjF7F1SM%3D&reserved=0>:
>     Spec restricts link destinations in formatted messages to
>     non-negative integers
>   * #434
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F434&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235508892&sdata=vEYvyz89aQv9zNO1%2BZjDPOFqqjSs3wNit%2BcDjfIvTM8%3D&reserved=0>:
>     Clarify that per-sub-rule metadata is not supported
>   * #435
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F435&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235518885&sdata=NfcTwmLRkR7qRpFB6V9Sw%2B%2F5L1FqNU9VMPN3gu85MT0%3D&reserved=0>:
>     Use "document" rather than "specification" as appropriate
>   * #436
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F436&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235518885&sdata=ZhJcM3WhC2uKmQEg%2FqMGd%2BDGFiRtH4KllSEPz543dvo%3D&reserved=0>:
>     Missing curly brace in example
>   * #437
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F437&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235528877&sdata=whEZR%2BKHsgRK2B4WVNk0uKjgfSNTRX2Ver4%2FmWLa7GQ%3D&reserved=0>:
>     Fix result.rule.description in schema
>   * #440
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F440&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235528877&sdata=3Pkfuvfgnuyag5uWGVmzLgZ6VE%2BxHSlV4EZ0Rc0x4f4%3D&reserved=0>:
>     Fix typo in 3.14.28 section title
>   * #444
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F444&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235538877&sdata=sAcl9blZbRkL33Ma3igZKtQkXGtZUhbh6heumeSZLgY%3D&reserved=0>:
>     Errors in K.4 Comprehensive SARIF file
>   * #445
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F445&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235538877&sdata=jqDyT7sAhFv0teMjILzmyYnBKAjVqKvcYEqwBLxkTms%3D&reserved=0>:
>     Editorial issue in invocation.startTimeUtc and invocation.endTimeUtc
>     descriptions
>   * #446
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F446&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235548867&sdata=AotN3MuzfR%2FidT52UG8DA%2FtpBP7LScuew004E6AZnTw%3D&reserved=0>:
>     Invalid Hyperlinks found
>   * #448
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F448&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235558862&sdata=FJs8JrUQe%2FQWkVQ4r6dw0qgfTiCWKZyGzJ1fJFX0vqg%3D&reserved=0>:
>     Duplication of v2.1.0 JSON schema
>   * #449
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F449&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235558862&sdata=fEBLn9OmDkTGp1MeeCzruIXgz9sEIBR%2BOg0igGnISw4%3D&reserved=0>:
>     Suppression "status" property mis-spelled as "state" in the schema
>   * #450
>     <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foasis-tcs%2Fsarif-spec%2Fissues%2F450&data=02%7C01%7Cv-lgold%40microsoft.com%7Cc4876d2c8b604602f43e08d7b17efadb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637173031235568855&sdata=g5%2B1B8bR3DdIW%2BIEFyHhlZ9lJHvD6jPrNxVAOmgMxxI%3D&reserved=0>:
>     Use of obsolete baselineState.existing property persists in spec
>   * #451 <https://github.com/oasis-tcs/sarif-spec/issues/451>: Clarify
>     guidance on use of space in plain text messages
>
> Thanks,
>
> Larry