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

[WIP] Only watch config file that exists on the level of source files #1643

Merged
merged 1 commit into from
Apr 10, 2017
Merged

[WIP] Only watch config file that exists on the level of source files #1643

merged 1 commit into from
Apr 10, 2017

Conversation

nonnontrivial
Copy link

This addresses #1486.

Will now only watch a next.config.js that is on the level of the source files (adjacent to where /pages (or however named) lives).

WIP tag because the second requirement in the issue: "[that we watch] on a lower level than the source files" is unclear.

@leo, does this imply that all directories "on a lower level" should be traversed for next.config.js?

@leo
Copy link
Contributor

leo commented Apr 7, 2017

Sweet! That's how I meant it. But why are you going two levels up from inside the pages directory (../..) instead of just one? Wouldn't that mean we're landing outside the project dir?

@nonnontrivial
Copy link
Author

nonnontrivial commented Apr 7, 2017

Locally I had specified a pages directory in my next.config.js that was deeper than it should have been. Ugh! Fixed.

@leo
Copy link
Contributor

leo commented Apr 10, 2017

Sweet!

@leo
Copy link
Contributor

leo commented Apr 10, 2017

Thanks 😊

@leo leo merged commit c9bfba6 into vercel:master Apr 10, 2017
@nonnontrivial nonnontrivial deleted the config-file-scope branch April 10, 2017 17:08
@lock lock bot locked as resolved and limited conversation to collaborators Jan 18, 2019
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.

None yet

2 participants