Skip to content
This repository has been archived by the owner on Oct 22, 2022. It is now read-only.

Tracing #76

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Tracing #76

wants to merge 4 commits into from

Conversation

zrzka
Copy link
Contributor

@zrzka zrzka commented Apr 25, 2017

:tracing support described in #75

@moea
Copy link
Member

moea commented Apr 25, 2017

Thanks, will merge after giving this a try.

@zrzka
Copy link
Contributor Author

zrzka commented Apr 25, 2017

Okay. Then let me know when it will be pushed as SNAPSHOT and I will enable it in our lambda functions on dev to test it.

@moea
Copy link
Member

moea commented May 2, 2017

Just got back from a trip, will merge #77 first as it's a bugfix.

@moea
Copy link
Member

moea commented May 15, 2017

@zrzka I haven't forgotten about this - work has just been nuts, and because the automated test infrastructure isn't great (since a plugin like this is intrinsically difficult to test), I haven't been able to set time aside to put it through its paces. My plan is to merge & release this, and then take a look at whether recent versions of lein support plugins w/ an AWS SDK dependency (they couldn't previously, due to a conflict), which'd make code easier to test & remove the necessity for upgrading the AWS CLI when supporting new features.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants