Jared Short

Event type defined via query string (webhooks integration)

2

Using the header is okay, but it prevents me from just posting in arbitrary events from typical webhook systems. I would like to see a GET and POST based endpoints...

License is marked in the package.json as ISC, but that is default

1

It would be great to get a LICENSE file in the repo, just for clarity. Thanks!

Support role assumption from long term creds

2

Instead of encouraging privileged long term creds, we should encourage long term creds with permissions only to assume an IAM role. Even better if we can restrict them to only...


Add simple test suite

2

Easiest way to test - several example sls templates - run the plugin, compare generated image to known good image

https://github.com/mstefaniuk/graph-viz-d3-js/blob/master/README.md A browser version would be great for CI based build / review and could open up a rapid understanding of large topology. (Think multi-project applications) - zoom - search...

Handle "DependsOn" cloudformation attribute

1

sometimes in CFN the dependencies are not explicit via a Ref, so you need a "DependsOn" defined. This should be traced in the graph. http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-attribute-dependson.html

https://github.com/knsv/mermaid - web native - js only dependency

Different types of resources could have better edge labeling that makes the dependency more clear. Let's make this modular (per type / relationship perhaps?)

The icons currently default to the event based icon, but if it is a schedule based there is a different icon. We would need to look deeper in the object...