Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Shaape support #320

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

snecklifter
Copy link
Collaborator

Shaape fails on all environments and is no longer maintained [1]

  1. TypeError: 'int' object is not iterable christiangoltz/shaape#20

Shaape fails on all environments and is no longer maintained [1]

1. christiangoltz/shaape#20
@pepijnve
Copy link
Member

pepijnve commented Jan 8, 2021

I'm not inclined to remove the glue code even if we can no longer test it easily. Someone somewhere might be using this in a documentation pipeline so removing this would be a breaking change.

Perhaps on a 3.x branch we could do this.

@snecklifter
Copy link
Collaborator Author

@pepijnve sure, the linked issue indicated to me that it doesn't work on python 2.7 and I can't see it working on python 3 as the code hasn't been updated for over 6 years. I'll look at creating a branch for stuff like this perhaps.

@pepijnve pepijnve force-pushed the master branch 2 times, most recently from c91ef06 to adc02e0 Compare March 23, 2023 12:58
@pepijnve pepijnve changed the base branch from master to main February 10, 2024 13:33
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