Skip to content

Conversation

@cameronezell
Copy link

No description provided.

@lhaig
Copy link

lhaig commented Mar 26, 2020

I was going to do this but now I just have to add +1

Thanks @cameronezell

@avram avram mentioned this pull request Aug 27, 2020
@avram
Copy link
Contributor

avram commented Aug 27, 2020

@cameronezell @lhaig This doesn't test cleanly -- can you look at #161 and the failures there?

Closing in favor of #161 so we can track the test status cleanly

@avram avram closed this Aug 27, 2020
@cameronezell
Copy link
Author

@avram Hi, it looks like the FullLoader attribute isn't present in the PyYaml version specified in the requirements.txt file. I think bumping up that version should work or I could test just changing safe_load to load and see if it eliminates the error despite the missing attribute.

@avram
Copy link
Contributor

avram commented Aug 27, 2020

Those all sound like viable solutions. Happy to merge and a release a working version of this fix. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants