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

[datadog_sensitive_data_scanner_rule] Add priority level in rules #2376

Merged
merged 13 commits into from
Apr 25, 2024

Conversation

trosenblatt
Copy link
Contributor

@trosenblatt trosenblatt commented Apr 22, 2024

JIRA_TICKET This enable support for priority field in SDS Rules in datadog terraform provider

vinckama
vinckama previously approved these changes Apr 22, 2024
Copy link
Contributor

@vinckama vinckama left a comment

Choose a reason for hiding this comment

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

LGTM

@trosenblatt trosenblatt marked this pull request as ready for review April 22, 2024 13:38
@trosenblatt trosenblatt requested review from a team as code owners April 22, 2024 13:38
rtrieu
rtrieu previously approved these changes Apr 22, 2024
Copy link
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

Minor suggestion, but otherwise looks good.

docs/resources/sensitive_data_scanner_rule.md Outdated Show resolved Hide resolved
vinckama
vinckama previously approved these changes Apr 23, 2024
@skarimo skarimo changed the title [SDS] Add priority level in rules [datadog_sensitive_data_scanner_rule] Add priority level in rules Apr 23, 2024
CHANGELOG.md Outdated Show resolved Hide resolved
Remove changelog

Co-authored-by: skarimo <[email protected]>
@trosenblatt trosenblatt requested review from skarimo and rtrieu April 23, 2024 15:27
skarimo
skarimo previously approved these changes Apr 23, 2024
@skarimo skarimo self-requested a review April 23, 2024 15:35
rtrieu
rtrieu previously approved these changes Apr 23, 2024
Copy link
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

Same feedback.

@trosenblatt
Copy link
Contributor Author

Sorry @rtrieu I accepted your changed but the script to generated docs overrides it. I think it is good know

@trosenblatt trosenblatt requested a review from rtrieu April 23, 2024 15:49
@rtrieu
Copy link
Contributor

rtrieu commented Apr 23, 2024

Hi @trosenblatt I think it did it again 😅

@trosenblatt
Copy link
Contributor Author

@rtrieu what do you mean it did it again ? I replaced the is with being. Is there anything else ?

@rtrieu
Copy link
Contributor

rtrieu commented Apr 23, 2024

Hi @trosenblatt I meant that it looks like the script overrode my suggestion and review. I will approve again now!

rtrieu
rtrieu previously approved these changes Apr 23, 2024
Copy link
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

LGTM!

@skarimo skarimo merged commit 6daca2c into master Apr 25, 2024
10 checks passed
@skarimo skarimo deleted the trosenblatt/addSdsRulePriority branch April 25, 2024 14:51
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.

4 participants