Skip to content

Conversation

@JayjeetAtGithub
Copy link

@JayjeetAtGithub JayjeetAtGithub commented Apr 21, 2020

Add support for library:// URLs in spython build and pull #155

@vsoch
Copy link
Member

vsoch commented Apr 21, 2020

Looks like circle changed something to need extra permission to use the Orb, and it's buggy!
image

I'm seeing if I can fix this up.

@JayjeetAtGithub
Copy link
Author

@vsoch Yeah. Thanks a lot.

@vsoch
Copy link
Member

vsoch commented Apr 21, 2020

While we are waiting, please update the version.py (version in there) and a matching line in the CHANGELOG.md. If the circle issue doesnt resolve, I'll pull your changes and PR from my branch, in case that works.

@JayjeetAtGithub
Copy link
Author

Done @vsoch

@vsoch
Copy link
Member

vsoch commented Apr 21, 2020

okay, the page seems to load (and I have the setting checked) so it must have issue with running the test since you aren't an org member. I'm going to go ahead and pull the branch and open in a PR and see if that works. Your changes look great and spot on, btw, it's just good to get testing done! If this is a new issue with using Circle orbs I'll need to think about how to address this in the future.

image

Anyway stay tuned!

@vsoch vsoch mentioned this pull request Apr 21, 2020
@vsoch
Copy link
Member

vsoch commented Apr 21, 2020

Merged with #156, thanks @JayjeetAtGithub ! I'll get the release ready for pypi now.

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.

2 participants