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 easter eggs #3128

Closed
tiran opened this issue Oct 30, 2018 · 8 comments
Closed

Remove easter eggs #3128

tiran opened this issue Oct 30, 2018 · 8 comments

Comments

@tiran
Copy link

tiran commented Oct 30, 2018

I just noticed the tweet https://twitter.com/KumarVimal/status/1057149022346137600 . Apparently pipenv has an easter egg that shows a pumpkin around Halloween, see

pipenv/pipenv/core.py

Lines 73 to 93 in d069dff

if not PIPENV_HIDE_EMOJIS:
now = time.localtime()
# Halloween easter-egg.
if ((now.tm_mon == 10) and (now.tm_mday == 30)) or (
(now.tm_mon == 10) and (now.tm_mday == 31)
):
INSTALL_LABEL = "🎃 "
# Christmas easter-egg.
elif ((now.tm_mon == 12) and (now.tm_mday == 24)) or (
(now.tm_mon == 12) and (now.tm_mday == 25)
):
INSTALL_LABEL = "🎅 "
else:
INSTALL_LABEL = "🐍 "
INSTALL_LABEL2 = crayons.normal("☤ ", bold=True)
STARTING_LABEL = " "
else:
INSTALL_LABEL = " "
INSTALL_LABEL2 = " "
STARTING_LABEL = " "
# Enable shell completion.
. I'm really worried about this.

I'm aware that some people find easter eggs an amusing gimmick in software. However easter-eggs make software unpredictable and are a even potential security. This is especially true for easter eggs that are triggered by default. CPython has some easter eggs like import this, however they are never triggered either time, environment, or default. Pipenv strives to become a defacto tool for Python's packaging infrastructure. Please be more responsible and serious.

Bugs like Gimme Gimme Gimme have been caused by easter eggs.

@Conan-Kudo
Copy link

This is a huge overreaction. All it does is literally change which emoji is used during what time of year if emoji can be used.

It's a pretty safe alteration, and it's not worth killing an easter egg like this over it. VLC is another example of an application who does something like this. It just swaps out the icon for Christmas time with one that has Santa's hat on top.

Please be more responsible and serious.

Please understand that levity is important for people to enjoy what they do. Sucking the personality and life out of applications just means it's not worth developing them anymore.

@tiran
Copy link
Author

tiran commented Oct 30, 2018

You can't seriously compare pipenv to VLC. VLC is a user-facing application and dedicated to entertainment. Also VLC is never executed as privileged root user, too. On the other hand, pipenv also aims to be used in automated tasks and executed as root.

Do you want pipenv to be a fun, cute toy or a professional, serious, reliable application?

And no, I'm not asking for all easter eggs to die. It's depends on the circumstances. For some software, it can be ok. However time based easter eggs, easter eggs in automation tools, and especially easter eggs in software that runs as root, are a no-go. Just don't.

@Conan-Kudo
Copy link

Pipenv is not supposed to run as root either. If people are doing that, they're using it wrong.

Do you want pipenv to be a fun, cute toy or a professional, serious, reliable application?

I want my fun and cuteness along with reliability. This idea that professionalism implies killing all the fun out of these things has been ridiculous from day one.

VLC is a user-facing application and dedicated to entertainment.

VLC can be used for Serious Business(TM) too. It's often used for live transcoding and streaming, since it has an engine for it built-in. But that's beside the point.

@kennethreitz
Copy link
Contributor

The easter eggs stay.

@tiran
Copy link
Author

tiran commented Oct 30, 2018

@kennethreitz That's one way to kill a discussion. Not necessarily a polite way, though.

@kennethreitz
Copy link
Contributor

The discussion can continue.

@tiran
Copy link
Author

tiran commented Oct 30, 2018

Then keep the ticket open until the discussion has come to a conclusion.

@vstinner
Copy link

I wanted to check if the easter egg can cause Unicode encode errors at Halloween, but in fact, it doesn't work all days :-) See my bug report #3131.

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

No branches or pull requests

4 participants