Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

dlm-lifecycle-policy: Fixing readme typos #329

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

seansaleh
Copy link

Some variables didn't match the actual variable names, and ebs_target_tags needs to be a dict

  • Update the changelog
  • Make sure that modules and files are documented. This can be done inside the module and files.
  • Make sure that new modules directories contain a basic README.md file.
  • Make sure that the module is added to tests/main.tf
  • Make sure that the linting passes on CI.
  • Make sure that there is an up to date example for your code:
    - For new modules this would entail example code for how to use the module or some explanation in the module readme.
    - For new examples please provide a README explaining how to run the example. It's also ideal to provide a basic makefile to use the example as well.
  • Make sure that there is a manual CI trigger that can test the deployment.

Some variables didn't match the actual variable names, and `ebs_target_tags` needs to be a dict
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant