Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Address object in location (and other nodes) : https://github.com/oasis-tcs/sarif-spec/issues/302 #1323

Merged
merged 6 commits into from
Mar 7, 2019

Conversation

harleenkohli
Copy link
Contributor

No description provided.

@harleenkohli harleenkohli changed the title Address object in location (and other nodes) Address object in location (and other nodes) : https://github.com/oasis-tcs/sarif-spec/issues/302 Mar 2, 2019
"uniqueItems": false,
"default": [],
"items": {
"$ref": "#/definitions/address"
Copy link
Member

@michaelcfanning michaelcfanning Mar 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

address [](start = 47, length = 7)

make this externalPropertyFile #Resolved

@@ -560,6 +596,17 @@
}
},

"addresses": {
"description": "The addresses associated with external property files, if any.",
Copy link
Member

@michaelcfanning michaelcfanning Mar 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The addresses associated with external property files, if any. [](start = 36, length = 62)

This s/be

"An array of external property files containing run.addresses arrays to be merged with the root log file." #Resolved

@@ -1947,6 +1999,17 @@
}
},

"addresses": {
"description": "The addresses associated with this run instance, if any.",
Copy link
Member

@michaelcfanning michaelcfanning Mar 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The addresses [](start = 36, length = 13)

Just say "Addresses associated..." #Resolved

@michaelcfanning
Copy link
Member

michaelcfanning commented Mar 6, 2019

        MarkdownMessageMimeType = "text/markdown;variant=GFM";

duped #Resolved


Refers to: src/Sarif/NotYetAutoGenerated/Run.cs:194 in 36fd1c7. [](commit_id = 36fd1c7, deletion_comment = False)

Copy link
Member

@michaelcfanning michaelcfanning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@michaelcfanning michaelcfanning merged commit 37ababe into master Mar 7, 2019
@michaelcfanning michaelcfanning deleted the address-object-in-location branch March 7, 2019 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants