-
Notifications
You must be signed in to change notification settings - Fork 607
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
Set tags on log groups #1164
Set tags on log groups #1164
Conversation
We haven't been using a YAML formatter, but maybe we should! Which one do you use? Regardless, I think we should make a separate PR after this one if we do decide to reformat, to keep the diff in the PR more relevant |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job on this, using log_group_aws_tags
is a great idea!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thank you!
closes #1078
How do you format yaml's? My VSCode formats them like this.
checklist:
make test
andmake lint
summary.md
(view in gitbook after merging)