search

JulienPalard / Pipe

A Python library to use infix notation in Python
MIT License
1.95k stars 113 forks source link

Add how to install in README #66

Closed devRMA closed 2 years ago

devRMA commented 2 years ago

Fix the issue #61

JulienPalard commented 2 years ago

I often see this --upgrade/-U flag in the install line, but I don't really like it, as it's a bit of a lie, it's not the command to install, it's the command to install or upgrade, but I don't expect users reading this line to need the --ugrade part.

Is there a real use-case I am missing behind this?

devRMA commented 2 years ago

I often see this --upgrade/-U flag in the install line, but I don't really like it, as it's a bit of a lie, it's not the command to install, it's the command to install or upgrade, but I don't expect users reading this line to need the --ugrade part.

Is there a real use-case I am missing behind this?

I used as a base, the README of this repository. https://github.com/Rapptz/discord.py#installing I honestly don't know the difference between using the -U flag or not, I just put it, because in the discord.py repository, the installation example taught like that, but I can remove that part.

JulienPalard commented 2 years ago

Back from holidays!

As we both don't see a good reason to use -U, yes, please remove it.

If in the future we see a good reason to add it it can be added with a commit message explaining why it's added, which I prefer.

devRMA commented 2 years ago

Back from holidays!

As we both don't see a good reason to use -U, yes, please remove it.

If in the future we see a good reason to add it it can be added with a commit message explaining why it's added, which I prefer.

ok

JulienPalard commented 2 years ago

:+1: thanks for the PR!

And happy holidays!!