Hi @feng-tao, actually I though of these two concerns you’re pointing out:

1.  to my understanding, bootstrap-theme.css is maintained using npm. We should 
not make change to it in our codebase. I hesitated if I should fix from 
bootstrap-theme.css, but eventually gave up due to this reason.

2. Putting “warning” rather than “danger”: I also hesitated back and forth on 
this. Some Exceptions here are not causing system-wide trouble, like failed to 
import Variable. Putting “danger” category seems too much (available options 
are only “success”, “info”, “warning”, “danger”).

[ Full content available at: 
https://github.com/apache/incubator-airflow/pull/3903 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to