-
Notifications
You must be signed in to change notification settings - Fork 372
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
Clean up hy.contrib #1189
Clean up hy.contrib #1189
Conversation
I don't see why you'd put this in the standard library. I guess it could be useful for when you're maintaining a library and you want to change the name of a function or macro but keep the old name around for a while so people's code doesn't break immediately. But that's a pretty limited purpose.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a bit sad to see Botsbuildbots to go
I'm going to go and listen to Portal 2 OST
As a side note: what if we leave Botsbuildbots, but just remove it from the tests? Seems kind of silly that that's tested, but it's still a fun little easter egg. I personally kind of see it as akin to Python's |
65effb0
to
16de4ef
Compare
@kirbyfan64 Ugh, fine. |
This is an Easter egg whose tests add a dependency (requests) and cause a test failure if you don't have an active Internet connection.
It's undocumented and of dubious value.
Now we can put it into contrib.multi instead of its own module.
It's not tested, and sure enough, a glance at the code suggests that `case` and `switch` will evaluate their first argument once for every clause, which is unlikely to be desirable. I say remove it, but if anybody wants to fix and test and re-add `case` (and change it to a square-bracket syntax like `cond`), be my guest.
This seems to be specific to Flask, a web framework.
16de4ef
to
91b26d1
Compare
👍 happy to see the cleanup! |
(FWIW; I'm OK with removing |
Closes #1162.