Skip to content
This repository has been archived by the owner on Mar 26, 2018. It is now read-only.

feat(Gruntfile): switch to BrowserSync #1029

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

feat(Gruntfile): switch to BrowserSync #1029

wants to merge 2 commits into from

Conversation

felipemarcos
Copy link

Closes #764

livereload: {
bsFiles: {
src: [
'<%%= yeoman.app %>/**/*.html',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to be consistent, it should be '<%%= yeoman.app %>/{,*/}*.html'

@eddiemonge
Copy link
Member

will have to test this out but looks nice for the most part

@ghost
Copy link

ghost commented Apr 4, 2015

@eddiemonge you have been testing for 25 days

@stephanebachelier
Copy link

@supergray it's open source done on our extra time. So please, be kind.

@felipemarcos
Copy link
Author

Hi @eddiemonge, any news? Thanks!

@eddiemonge
Copy link
Member

not yet. soon though

@eddiemonge
Copy link
Member

  • resolve the merge conflicts
  • squash the commits

@arthurvr
Copy link
Member

We now also support Gulp so we should switch to browsersync in the gulpfile too.

@aeby
Copy link

aeby commented Aug 30, 2015

What do you think about adding the connect-history-api-fallback middleware to support HTML5 mode? Have a look at this discussion.

@eddiemonge
Copy link
Member

@aeby no thank you. This is for generating an angular app, not a server configuration. Not everyone wants to use HTML5Mode. There might be an option in the future though. Also, not the right place to ask the question as this is for this pull request.

DiscoPogo pushed a commit to DiscoPogo/generator-angular that referenced this pull request Nov 21, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch to BrowserSync
5 participants