{"action":"created","issue":{"url":"https://api.github.com/repos/dotnet-state-machine/stateless/issues/38","repository_url":"https://api.github.com/repos/dotnet-state-machine/stateless","labels_url":"https://api.github.com/repos/dotnet-state-machine/stateless/issues/38/labels{/name}","comments_url":"https://api.github.com/repos/dotnet-state-machine/stateless/issues/38/comments","events_url":"https://api.github.com/repos/dotnet-state-machine/stateless/issues/38/events","html_url":"https://github.com/dotnet-state-machine/stateless/pull/38","id":119000277,"number":38,"title":"DotGraph now renders OnEntry and OnExit actions with descriptions","user":{"login":"cmendible","id":266546,"avatar_url":"https://avatars.githubusercontent.com/u/266546?v=3","gravatar_id":"","url":"https://api.github.com/users/cmendible","html_url":"https://github.com/cmendible","followers_url":"https://api.github.com/users/cmendible/followers","following_url":"https://api.github.com/users/cmendible/following{/other_user}","gists_url":"https://api.github.com/users/cmendible/gists{/gist_id}","starred_url":"https://api.github.com/users/cmendible/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmendible/subscriptions","organizations_url":"https://api.github.com/users/cmendible/orgs","repos_url":"https://api.github.com/users/cmendible/repos","events_url":"https://api.github.com/users/cmendible/events{/privacy}","received_events_url":"https://api.github.com/users/cmendible/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-11-26T09:01:14Z","updated_at":"2016-03-03T10:06:30Z","closed_at":"2015-12-10T10:51:49Z","pull_request":{"url":"https://api.github.com/repos/dotnet-state-machine/stateless/pulls/38","html_url":"https://github.com/dotnet-state-machine/stateless/pull/38","diff_url":"https://github.com/dotnet-state-machine/stateless/pull/38.diff","patch_url":"https://github.com/dotnet-state-machine/stateless/pull/38.patch"},"body":"Hi, I've been using stateless for several months now and one of my clients was missing meaningful descriptions for *Trigger Guards*, *OnEntry* and *OnExit* actions within the *DotGraph* feature.\r\n\r\nFor instance now it's possible to write the following code (which removes the compiler generated method names such as *<test>b__5_0* from the resulting string):\r\n\r\n    machine.Configure(State.A)\r\n        .OnEntry(() => { }, \"Just entered A\")\r\n        .PermitIf(Trigger.X, State.B, () => true, \"description\");\r\n\r\nAnd get the following *DotGraph* string:\r\n\r\n    digraph {\r\n     A -> B [label=\"X [description]\"];\r\n     node [shape=box];\r\n     A -> \"Just entered A\" [label=\"On Entry\" style=dotted];\r\n    }\r\n\r\nHope you like it as much as our client  does and thanks for sharing this great code!!!\r\nCarlos\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/dotnet-state-machine/stateless/issues/comments/191691247","html_url":"https://github.com/dotnet-state-machine/stateless/pull/38#issuecomment-191691247","issue_url":"https://api.github.com/repos/dotnet-state-machine/stateless/issues/38","id":191691247,"user":{"login":"tlk","id":90870,"avatar_url":"https://avatars.githubusercontent.com/u/90870?v=3","gravatar_id":"","url":"https://api.github.com/users/tlk","html_url":"https://github.com/tlk","followers_url":"https://api.github.com/users/tlk/followers","following_url":"https://api.github.com/users/tlk/following{/other_user}","gists_url":"https://api.github.com/users/tlk/gists{/gist_id}","starred_url":"https://api.github.com/users/tlk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tlk/subscriptions","organizations_url":"https://api.github.com/users/tlk/orgs","repos_url":"https://api.github.com/users/tlk/repos","events_url":"https://api.github.com/users/tlk/events{/privacy}","received_events_url":"https://api.github.com/users/tlk/received_events","type":"User","site_admin":false},"created_at":"2016-03-03T10:06:29Z","updated_at":"2016-03-03T10:06:29Z","body":"I see two approaches to remedy this:\r\n1) add overload methods to be backwards compatible\r\n2) bump the library semver (not sure how that is done with NuGet)\r\n\r\nIt looks like some people have been struggling with this PR due to the usage of new C# syntax:\r\n- https://github.com/vilcans/stateless/commit/ae14ecd7b65fc1382e6618a40d22c229d2d13ec3\r\n- https://github.com/UnitedTraders/stateless/commit/5cc0fe5b7c8a5696b729d3f76543e2f7887dbb1a\r\n\r\n@cmendible @nblumhardt Thoughts?"}}