Skip to content

Conversation

@imnotjames
Copy link
Contributor

@imnotjames imnotjames commented Oct 21, 2019

Implements a "watcher" on files being added and removed when --live is specified in the command line arguments.

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

  • I acknowledge that all my contributions will be made under the project's license.

@imnotjames
Copy link
Contributor Author

imnotjames commented Oct 21, 2019

This is a second attempt at #53 now that I understand the problem a bit better :)

@imnotjames imnotjames changed the title Feature/53/watch mode parameter feat: support loading of new files automatically with --live Oct 21, 2019
@imnotjames imnotjames marked this pull request as ready for review October 31, 2019 03:58
Copy link
Contributor

@dkundel dkundel left a comment

Choose a reason for hiding this comment

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

Hi @imnotjames! Thank you so much for this contribution and sorry for the late reply. I added a few comments :)

this utilizes the chokidar watcher to watch all JS files
under the functions directory and for all items within the
assets directories.

this also uses the lodash debouncer to prevent multiple files
changing in quick succession (250ms) from causing too many
reloads.
@imnotjames imnotjames force-pushed the feature/53/watch-mode-parameter branch from bb0166e to 1a2d83f Compare November 2, 2019 02:51
@imnotjames imnotjames requested a review from dkundel November 2, 2019 02:51
@dkundel dkundel merged commit 955efbe into twilio-labs:master Jan 14, 2020
@pika-ci
Copy link

pika-ci bot commented Jan 14, 2020

🚀 This PR has been merged! Once a new release is created, any changes will become available on npm. Until then, you can load and install it directly from the Pika CDN:

npm install https://cdn.pika.dev/twilio-run/master

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.

2 participants