undefined method `run_context' for Chef:Class about chef-slack_handler

[cknowles]

I think that was changed in #22 but not sure what the minimum version should be. Using v12 here.

[martinb3]

We've definitely released the changes in #22.

[martinb3]

Chef 12.x is about 1.5 years old, so I've only been testing on it. It wouldn't surprise me if there was something broken in Chef 11.x. I'd be happy to merge PRs to add compatibility back, if so. 👍

[glensc]

i was investigating this, and i fixed it locally by saving run_context from report() locally as @run_context, instead of "asking" it from chef internals. this also involved moving @util creation to report method.

would you accept such rewrite as PR?

[martinb3]

@glensc Would you consider building a branch that exercises the issue? I'd like to understand further what's happening here. The chef run_context object was definitely around in Chef 11.x.

[glensc]

yes, but not in exact place you're taking it now. and i feel it's using it's internal structures is unreliable. and storing it as instance property @run_context makes rest of the code also more readable (don't need to pass it around between methods)

ok, i'll try to clean it up and show the branch, but that tomorrow, hopefully.

[kjwierenga]

This is currently blocking me from using chef-slack_handler with Chef v11.12.4

[kjwierenga]

Thanks @glensc. I actually arrived at almost the same solution before seeing your commit 😄

[glensc]

💃

[glensc]

closing as #35 was merged