elastic / beats-dashboards

DEPRECATED. Moved to https://github.com/elastic/beats. Please use the new repository to add new issues.
Other
196 stars 73 forks source link

Update script with curly braces in the script #86

Closed radoondas closed 8 years ago

radoondas commented 8 years ago

This is simple update which adds 'curly-braces' Extending: elastic/beats-dashboards#83

andrewkroh commented 8 years ago

Except for the one comment, LGTM.

It seems like #83 can be closed because the s/name/NAME/g change is implemented here.

ruflin commented 8 years ago

LGTM except for the part discovered by @andrewkroh . @radoondas Let us know when you pushed the change as Github does not send notifications for new commits.

radoondas commented 8 years ago

@ruflin I have just pushed fix to github. Could someone please double check?

ruflin commented 8 years ago

@radoondas LGTM. Merged.