-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[now-static-build][frameworks][examples] Fixes examples and adjust frameworks #3584
Conversation
This pull request is being automatically deployed with ZEIT Now (learn more). 🔍 Inspect: https://zeit.co/zeit/now/lw2ma68gf |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now it makes sense!
@@ -145,6 +168,7 @@ export const frameworks: Framework[] = [ | |||
slug: 'angular', | |||
dependency: '@angular/cli', | |||
minNodeRange: '10.x', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can remove minNodeRange
since it doesn't do anything anymore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done 👍
getOutputDirName: async () => 'public', | ||
}, | ||
{ | ||
name: 'Middleman', | ||
slug: 'middleman', | ||
buildCommand: 'bundle exec middleman build', | ||
devCommand: 'bundle exec middleman server -p $PORT', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was the dev command removed for middleman?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@styfle all devCommands were removed since they are part of @now/frameworks`, so they are not really removed, they'll just be added at a later time
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AndyBitz Then why did you add devCommand: 'nuxt',
above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@styfle Good catch, that was a mistake
now
file
withpath
inframeworks.json
@now/static-build
uses thepackage.json
script if present instead of the framework provided one