lcdporto / boxing-app

Boxing's project Web App made with Angular and Angular Material
MIT License
3 stars 2 forks source link

git Ignore /app/index.html #9

Closed ighormartins closed 8 years ago

ighormartins commented 8 years ago

When gulp resolves the file injecting, it changes app/index.html which causes it to be identified as a "modified" file. Since everyone will have to use the gulp inject, it'll cause a lot of conflicts between developers. How can we keep using the gulp inject and avoid this behavior?

P.S. @boda49 I changed the issue description to better explain the problem. Please tell me if it's not right.

ighormartins commented 8 years ago

We should consider it as a bug since it's not behaving in the way it should. We have to find a solution for this. I'm opening it for discussion

ighormartins commented 8 years ago

After some thinking and discussion with @ricardolobo, It doesn't seem to be a bug. With or without auto-inject, the index.html would be changed whenever someone is developing a new feature. Conflicts here will be normal, and should be fixed before merging. Closing this issue by now.