Skip to content

Commit

Permalink
#137: Remove 'screenshot' file role.
Browse files Browse the repository at this point in the history
  • Loading branch information
Larry Golding committed May 3, 2018
1 parent 31eea52 commit c744bf0
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
Binary file modified Documents/ProvisionalDrafts/sarif-v2.0-wd02-provisional.docx
Binary file not shown.
4 changes: 4 additions & 0 deletions Documents/Schema changes between SARIF v1 and v2.md
Expand Up @@ -626,6 +626,10 @@ These overrides are noted on the earlier change.
- Add a `regions` property of type `region[]`, minItems 1, unique.
- Add a `rectangles` property of type `rectangle[]`, minItems 1, unique.

In the `file` object:

- Remove the enumerated value `"screenshot"` from the `roles` property.

## Changes to review at TC #16

- [Issue #139](https://github.com/oasis-tcs/sarif-spec/issues/139): "Don't require codeFlowLocation.location"
Expand Down
1 change: 0 additions & 1 deletion Schemata/sarif-schema.json
Expand Up @@ -390,7 +390,6 @@
"attachment",
"responseFile",
"resultFile",
"screenshot",
"standardStream",
"traceFile"
]
Expand Down

0 comments on commit c744bf0

Please sign in to comment.