Skip to content

Conversation

@c4urself
Copy link

No description provided.

Copy link

@tjanez tjanez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would also suggest updating the "⚠️ Current status of this project" part of the README since people will encounter it both, on GitHub and PyPI.

Copy link

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should all code be removed as well? At least the tests are failing now.

@c4urself
Copy link
Author

@ekohl -- we can certainly do that, FWIW I'm not a maintainer on this project, but I can include it in the PR if you are able to merge?

@ekohl
Copy link

ekohl commented May 21, 2020

Looking at my review icon (which is gray rather than green) I think I'm unable to.

@tjanez
Copy link

tjanez commented May 21, 2020

I think we could ask @peritus to review and merge this branch after it's ready.

@peritus
Copy link
Owner

peritus commented May 21, 2020

Sure. Can do that.

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.

4 participants