Vince Foley
Vince Foley
I like the look of the old copper still.. Some of them have cool coils... Could make for a nice inspiration, both shape and color...
I submitted a PR that fixes this: https://github.com/vishnevskiy/ElixirSublime/pull/36
Hmm.... this "argument overloading" is a very interesting idea, I think it might be worth analyzing it's implications a bit more - it doesn't look like any of the other...
Ok, so maybe make this configurable? Or perhaps this is a major version bump?
Another possibility is to make this "forward compatible" by using `rtm.connect` doing all the init lookups that `rtm.start` used to include...
Here's the changes I made in a branch to make this work.. https://github.com/binaryseed/Elixir-Slack/commit/e9a9395f79e0d58e3ba2dc7811f8dea33bfc9a50
And to initialize `channels` for example, I do this: ```elixir def handle_connect(slack, state) do channels = Slack.Web.Channels.list(%{token: slack.token}) |> Map.get(:channels) |> Map.new(&{&1.id, &1}) slack = slack |> Map.put(:channels, channels) {:ok,...
Here we go: https://github.com/BlakeWilliams/Elixir-Slack/pull/235 Likely need to do some docs tweaks alongside it...
I'm interested to learn more about the "hooks" idea... One potential use case that is just a step beyond what `telemetry` provides is the ability to add Distributed Tracing instrumentation...