search

cloudposse / terraform-aws-elasticsearch

Terraform module to provision an Elasticsearch cluster with built-in integrations with Kibana and Logstash.
https://cloudposse.com/accelerate
Apache License 2.0
216 stars 231 forks source link

clarification to readme #127

Open pfeifferj opened 2 years ago

pfeifferj commented 2 years ago

Describe the Bug

The usage section says

The table below correctly indicates which inputs are required.

This is a bit misleading as the example right underneath this text includes non-required inputs. Adding a link to the relevant section to "The table below" would make this clearer.

I tried to make the change myself and open an MR, however it seems the readme was manually edited -- instead of editing the README.yaml and using the build-harness