Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

Create data model for Web Security Scanner #559

Merged
merged 2 commits into from
Dec 21, 2022

Conversation

Nick-NCSU
Copy link
Contributor

No description provided.

@Nick-NCSU Nick-NCSU added the patch Increment the patch version when merged label Nov 28, 2022
@Nick-NCSU Nick-NCSU requested a review from a team as a code owner November 28, 2022 13:54
{
resourceName: 'Scan Run',
_type: 'google_cloud_scan_run',
_class: ['Finding'],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not a great class for this but not sure which one would be better.

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't really have a better alternative. If no one else has a complaint, it's probably fine.

},
],
relationships: [],
dependsOn: [],
Copy link
Contributor

Choose a reason for hiding this comment

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

Not as familiar with the graph-google-cloud project, but just making sure, there's not service-level entity this would be dependent on?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not that I am aware of, I am also not too familiar but from what I could find it didn't need any

@Nick-NCSU Nick-NCSU merged commit 22c35ca into main Dec 21, 2022
@Nick-NCSU Nick-NCSU deleted the int-5822-data-model-web-security-scanner branch December 21, 2022 15:14
@j1-internal-automation
Copy link
Collaborator

🚀 PR was released in v2.20.2 🚀

@j1-internal-automation j1-internal-automation added the released This issue/pull request has been released. label Jan 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
patch Increment the patch version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants