Jim Ladd

Results 16 comments of Jim Ladd

TLDR - basically, everything makes sense / sounds good 👍 . Appreciate you confirming the current behavior and fleshing out what the changes would entail. ---- The slightly longer response.....

@didoesdigital Definitely interested, but pretty short on cycles at the moment (just had baby #2) and also a level zero javascript n00b. Would love to dig into the code at...

Validation appears to be [off by default](https://github.com/pycontribs/jenkinsapi/blob/master/jenkinsapi/artifact.py#L34), but `self._verify_download` will get called regardless. I would expect `save` to do something like [this](https://github.com/jladdjr/jenkinsapi/commit/4139e19b7adc9b616d4b041a048aa8e64526368a) instead.

Also noticed that pypandoc is the only requirement listed as being outdated: https://requires.io/github/jlaska/pytest-github/requirements/?branch=master

> In your case, it seems you're missing https://pandoc.org itself I did do a `brew install pandoc` (after originally seeing an error suggesting that I was missing pandoc) and confirmed...

Hi back, I am currently swamped at work and don't have room to go back and get more details on this right now. I know that's not very helpful, but...

@wenottingham - any concerns about extra_vars potentially including sensitive data? If not, we can add this back in

While being able to show survey values sounds interesting, I've been trying to stay away from anything that would surface sensitive information (even an encrypted or hashed version). I'd vote...

So, what's the verdict for _this_ issue? @ryanpetrello @unlikelyzero et al, should we close?