-
Notifications
You must be signed in to change notification settings - Fork 6
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
gulp notify #17
Comments
@akfzambrana vc que colocou o notifier? ou estou enganado |
@woliveiras mencionou em #15 (comment) "Acho que o Notify pode até ser removido. Não sei se faz sentido pra vocês remover. Eu não uso por que não curto eles. kkkkk""" |
quem foi q implementou o notify? deve ter utilidad sim, só oq me incomoda, é o pipoco d qse 100 notifications q me popula, qdo rodo uma task d build por exemplo, o ideal seria rodar 1 notify por task, ou ao final d um watch (q roda um lot d tasks talvez) daí, se for realmente util, além d termos d corrigir esta qtd d notifications, sugiro tbm uma flag pra habilitar ou dsabilitar as notifications (ja q elas rolam pelo notifaction center no mac, e acredito q growl e outros d outros SO's) |
Sucesso mano. Eu não uso por que, normalmente, mando rodar o build e espero o erro no console. Normalmente é rápido o build no Gulp. Ainda mais num projeto simples. |
mas é útil qdo tamo trampando no code, q dawe o watch faz td q tem q fazer, conforme se mexe. mas tipo, o terminal ja notifica la td (por logs), o notification do SO acredito q seja opcional, sei lá, pod ter alguem q prefira tais feedbacks atrelados ao notification do SO |
Sim sim |
ahhh, tem mtos notifies rolando, sugiro ajustarmos ou removermos.
exemplo, ao rodar uma tarefa de build, (testei nessa PR #15), rola quase 100 notifications acumulados
uma idéia, rodar notifies somente se for passada uma flat, exemplo:
gulp server --notifier
ou se o padrão é notificar, desabilitar
gulp server --no-notifier
mas é apenas uma idéia, só desejo afinarmos o notifier, daí o motivo dessa issue pra discutirmos um bom approach
The text was updated successfully, but these errors were encountered: