Ryan Walls

Feature request: clickable links in notifications

2

Would love to be able to add a clickable link to notifications. Not sure how this would be implemented cross-platform, but toast on windows 10 has the capability (see https://github.com/go-toast/toast)....

LexicalPreservingPrinter indentation is broken when new field is added first in class

4

When adding a field to the top of a type that has fields already defined, the `LexicalPreservingPrinter` single spaces the inserted field and the first field that follows it. Then...

LexicalPreservingPrinter inserts extra line break when adding an import at index 0

1

When inserting a new import declaration at index 0, I would expect the `LexicalPreservingPrinter` to match the formatting of the other imports and not add an extra line break. The...

Feature request: Be able to set start and complete listeners for activities in production code

8

In tests you can use `SetOnActivityCompletedListener` and `SetOnActivityStartedListener`. Seems like it would be nice to be able to use that paradigm in production code as well. I have generic start/cleanup...

exec.Cmd gives you access to the process which you can then kill. Would be great if we could do a cmd.Interrupt() or cmd.Kill()

Feature request: Create branch or repo using fluentd driver instead of tailing logs

4

Would love to see an example where containers on a k8s cluster are set to use the fluentd logging driver.

Provide better custom logger example in README

1

## What was changed Updated the README with a more helpful logging example. ## Why? The current example did not make it clear how to "feed the logger" into Temporal....

https://github.com/Masterminds/glide is my favorite

Golang package naming conventions do not include underscores. Before this repo gets much use, would be nice to rename to `ecsstate`.

When I add the dependency to this project, I get a compilation error: ``` github.com/jhspaybar/ecs_state/ecs_state.go:47: cannot use db (type *gorm.DB) as type gorm.DB in field value ``` Have you guys...