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

Update examples to remove configuration #423

Merged
merged 124 commits into from Jul 19, 2019

Conversation

@msweeneydev
Copy link
Member

commented Jul 18, 2019

This PR updates examples to remove configuration.

leo and others added some commits Jun 27, 2019

@now

This comment has been minimized.

Copy link

commented on 42e833f Jul 1, 2019

Deployment failed with the following error:

You defined 3 builds that did not match any source files (please ensure they are NOT defined in .nowignore):
@now

This comment has been minimized.

Copy link

commented on 92f31fa Jul 1, 2019

Deployment failed with the following error:

You defined 3 builds that did not match any source files (please ensure they are NOT defined in .nowignore):
@now

This comment has been minimized.

Copy link

commented on 929fb21 Jul 1, 2019

Deployment failed with the following error:

You defined 26 builds that did not match any source files (please ensure they are NOT defined in .nowignore):
@now

This comment has been minimized.

Copy link

commented on af95501 Jul 1, 2019

Deployment failed with the following error:

You defined 26 builds that did not match any source files (please ensure they are NOT defined in .nowignore):
@now

This comment has been minimized.

Copy link

commented on fb063a5 Jul 1, 2019

Deployment failed with the following error:

You defined 26 builds that did not match any source files (please ensure they are NOT defined in .nowignore):

msweeneydev added some commits Jul 1, 2019

@leo leo requested review from timothyis and removed request for styfle, Timer, timothyis, matheuss and TejasQ Jul 19, 2019

@leo

leo approved these changes Jul 19, 2019

Done

@skllcrn skllcrn self-requested a review Jul 19, 2019

@msweeneydev msweeneydev merged commit bdc40cc into master Jul 19, 2019

1 check passed

now Deployment has completed
Details

@msweeneydev msweeneydev deleted the zero-config branch Jul 19, 2019

@dolphin278

This comment has been minimized.

Copy link

commented Jul 27, 2019

May I ask a question – why do you remove all now.json files and monorepo example completely? Will there be any new configuration mechanism instead of now.json or something, that renders now.json irrelevant?

@rauchg

This comment has been minimized.

Copy link
Contributor

commented Jul 27, 2019

Hey @dolphin278 – we don't think people should need any configuration to get started, so we came up with a convention.

Furthermore, all the examples that end in +functions fit our definition of monorepo. All the files contained in api/ become functions (in whatever language you specify), and the top level directory is the frontend of your app.

@dolphin278

This comment has been minimized.

Copy link

commented Jul 27, 2019

@rauchg thank you for explanation. Looks like major simplification for app development.

Does it mean that now.json-based routing in it's current form will be used only for specific use cases, like setting up cache headers, static redirects, etc (besides project-wide settings like aliases, regions, etc).?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
You can’t perform that action at this time.