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

move scripts to sh files [suggestion] #15

Merged
merged 1 commit into from
Jul 8, 2016

Conversation

darlanmendonca
Copy link
Contributor

tbm corrigi request pra smooth-scroll.js

@ldepaiva
Copy link
Contributor

ldepaiva commented Jul 8, 2016

Muito bom 👍

@darlanmendonca
Copy link
Contributor Author

:D nice,

as tasks do gulp (build e default) tbm existiem, coexistindo, ou seja

gulp => roda a task default (tbm possível por gulp default
gulp build roda as tasks pro build atual
npm s starta a tarefa default do gulp
npm run build roda a tarefa build do gulp

@darlanmendonca
Copy link
Contributor Author

ahh, vi q tem layout quebrado, vou averiguar se n quebrei nda, ou se ja tinha algo quebrado (to falando d layout, tem uns background q to axando, quem puder baixar a branch e dar um help :D

@ldepaiva
Copy link
Contributor

ldepaiva commented Jul 8, 2016

Percebi também que o layout do branch do gh-pages esta diferente do master, pode ter sido algum hard coded direto em gh-pages

@darlanmendonca darlanmendonca mentioned this pull request Jul 8, 2016
@woliveiras
Copy link
Member

Boa mano.

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

@woliveiras woliveiras merged commit 309e501 into abc-dev:master Jul 8, 2016
@darlanmendonca
Copy link
Contributor Author

@LucasAntoniassi eu tentei pela master, antes do merge e tava igual, mas axo q é alguma caca minha aki sim, vou a noite ver c calma e arrumamos

@woliveiras
Copy link
Member

Boa @darlanmendonca

Aqui parece estar certinho. Onde tem problema @LucasAntoniassi?

Manda uns print ae pros irmão. 😄

@darlanmendonca
Copy link
Contributor Author

vamos pra essa issue @woliveiras e @LucasAntoniassi

#19

@woliveiras
Copy link
Member

Booooa, monstro!

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

Successfully merging this pull request may close these issues.

3 participants