potiuk commented on PR #23335:
URL: https://github.com/apache/airflow/pull/23335#issuecomment-1112611198
> Case 3: Prompting for a decision when auxiliary actions should maybe be
taken. 👎 No, this is bad UX.
I Disagree. I want to make users to upgrade when they can. To be annoying
when they did not upgrade but continue nagging them . This is the goal. If you
have another proposal how to achieve that - I am all ears. Warnings printed
without annoying delays do not work. People ignore them.
> Power users should be able to set behavior using environment variables or
flags (see case 2) - this is exactly how docker build --no-cache works, and
they shouldn't be delayed by a timeout with a default answer - that just adds
needless delay to their day.
This is precisely what I want. I am targetting this for "non-power users".
Power users already have an option to skip the delay - they can add ``--answer
no``. So there is an escape hatch for power users to run stuff without delay.
I want "no upgrade" to be annoying for casual users when they need to
upgrade. But I do not want them to break them the workflow. This is for example
what NPM does and I really lilke it when I saw it (this is where the idea of
timout came from):
```
================================================================================
================================================================================
DEPRECATION WARNING
Node.js 10.x is no longer actively supported!
You will not receive security or critical stability updates for this
version.
You should migrate to a supported version of Node.js as soon as possible.
Use the installation script that corresponds to the version of Node.js you
wish to install. e.g.
* https://deb.nodesource.com/setup_12.x — Node.js 12 LTS "Erbium"
* https://deb.nodesource.com/setup_14.x — Node.js 14 LTS "Fermium"
(recommended)
* https://deb.nodesource.com/setup_16.x — Node.js 16 "Gallium"
Please see https://github.com/nodejs/Release for details about which
version may be appropriate for you.
The NodeSource Node.js distributions repository contains
information both about supported versions of Node.js and supported Linux
distributions. To learn more about usage, see the repository:
https://github.com/nodesource/distributions
================================================================================
================================================================================
Continuing in 20 seconds ...
```
If we are confident enough to take action after a timeout, then we should be
confident enough to just emit an info or warning message (and use Rich's colors
for that! 😄) and carry on with that action without any delay.
This is not 0/1 . People will ignore it mostly. As a maintainer, I want
"no-upgrade" to be annoying and make the people eventually click. Po
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
