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.

Sign up for GitHub

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

Jump to bottom

Update README.md #8

Merged
merged 1 commit into from
Dec 26, 2023
Merged

Update README.md #8

merged 1 commit into from
Dec 26, 2023

Conversation

royfur
Copy link
Contributor

@royfur royfur commented Dec 26, 2023

Update readme for instruction on the CSV integration

Description

Fixes #

How Has This Been Tested?

Checklist:

  • I have updated the versions in the SemanticVersion in template.yaml
  • I have updated the CHANGELOG.md
  • I have created necessary PR to Terraform Module Repository (https://github.com/coralogix/terraform-coralogix-aws) if needed
  • This change does not affect any particular component (e.g. it's CI or docs change)

@royfur
Update README.md
dd49e85 
Update readme for instruction on the CSV integration
@royfur royfur requested review from a team as code owners December 26, 2023 09:01
frank-ang
frank-ang approved these changes Dec 26, 2023
View reviewed changes
Copy link

@frank-ang frank-ang left a comment

Choose a reason for hiding this comment

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

LGTM.

@royfur royfur merged commit 73c8212 into master Dec 26, 2023
3 checks passed
@royfur royfur deleted the royfur-patch-1 branch December 26, 2023 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@frank-ang frank-ang frank-ang approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@royfur @frank-ang