Skip to content

allow loading of external config file - #7

Open
armandopadilla wants to merge 1 commit into
trailsjs:masterfrom
armandopadilla:master
Open

armandopadilla wants to merge 1 commit into
trailsjs:masterfrom
armandopadilla:master

Conversation

@armandopadilla

Copy link
Copy Markdown

The change allows users to load an external autoreload config. If non is provided the default one is used. Essentially, gives the user more control of this pack's underlining behavior without having to touch the npm-module directly.

How to use

  1. Create new file within your trails app. config/autoreload.js
  2. Add content

module.exports = {
cwd: '/srv/project',
ignoreInitial: true,
usePolling: true
}

  1. Reload app.

@scott-wyatt

scott-wyatt commented Jul 17, 2016 •

Copy link
Copy Markdown
Member

+1 for this functionality. Are you able to write a test for this as well? Also, please follow the eslint guidelines so that this passes the PR CI tests.

@armandopadilla

Copy link
Copy Markdown
Author

will do and on it!

@disciple-dev disciple-dev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Need to remove semicolons on lines 39 & 42 for CI tests to pass.

@scott-wyatt

scott-wyatt commented Oct 12, 2016 •

Copy link
Copy Markdown
Member

This functionality might be solved by trailsjs/trails#236 or trailsjs/trailpack#39

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