search

netlify / netlifyctl

MIT License
176 stars 34 forks source link

Replace `conf.Settings.Path` with `conf.Build.publish` #57

Closed zanona closed 7 years ago

zanona commented 7 years ago

Hey guys, the previous version of netlify.toml has:

[build]
  publish = 'dist/'

But netlifyctl now uses:

[build]
  publish = 'dist/'
[settings]
  path = 'dist/'

Perhaps we could streamline this to avoid redundancy and confusion?

https://github.com/netlify/netlifyctl/blob/e4a6a4d92824cf98e46ef22f26c7ab472b6d292d/commands/deploy/deploy.go#L104-L119

calavera commented 7 years ago

This is fixed in master now.