Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Add tags #59

Merged
merged 1 commit into from
Jun 11, 2024
Merged

Add tags #59

merged 1 commit into from
Jun 11, 2024

Conversation

rtest12
Copy link
Contributor

@rtest12 rtest12 commented May 30, 2024 •
edited
Loading

what

  • Use tags = module.this.tags on the notify_slack resource.

why

  • Enhanced manageability: Tags enable better resource management, cost tracking, and security strategies.
  • Consistency: Tagging the notify_slack resource provides consistency across all project resources.
  • Discovery: Tags assist in faster resource locating when filtering or searching.

@rtest12 rtest12 requested review from a team as code owners May 30, 2024 10:46
@mergify mergify bot added the triage Needs triage label May 30, 2024
@goruha
Copy link
Member

goruha commented May 31, 2024

/terratest

@rtest12
Copy link
Contributor Author

rtest12 commented Jun 11, 2024

@johncblandii @gberenice @Gowiem @jamengual @joe-niland @korenyoni @florian0410
ptl

@mergify mergify bot removed the triage Needs triage label Jun 11, 2024
@gberenice gberenice merged commit 8902df5 into cloudposse:main Jun 11, 2024
12 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@jamengual jamengual jamengual approved these changes

@johncblandii johncblandii Awaiting requested review from johncblandii johncblandii is a code owner automatically assigned from cloudposse/contributors

@gberenice gberenice Awaiting requested review from gberenice gberenice is a code owner automatically assigned from cloudposse/contributors

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

Successfully merging this pull request may close these issues.
4 participants
@rtest12 @goruha @jamengual @gberenice