search

frictionlessdata / datapackage-js

A JavaScript library for working with Data Package.
http://frictionlessdata.io/
MIT License
43 stars 15 forks source link

Consider using options argument instead of individual parameters #55

Closed roll closed 7 years ago

roll commented 7 years ago

Overview

For now it's e.g.:

constructor(descriptor, profile = 'base', raiseInvalid = true,  remoteProfiles = false, basePath = '')

but we could use:

constructor(descriptor, {profile = 'base', raiseInvalid = true,  remoteProfiles = false, basePath = ''})
new Datapackage(descriptor, {remoteProfiles: true})
dumyan commented 7 years ago

I'm in favor of using options arguments. This can be also implemented alongside the individual parameters.