-
Notifications
You must be signed in to change notification settings - Fork 160
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
pair configuration: collections = null
is not a good default.
#328
Comments
On Wed, Feb 03, 2016 at 02:05:29PM -0800, Markus Unterwaditzer wrote:
Since
|
Yes, but...
|
On Thu, Feb 04, 2016 at 11:33:08AM -0800, Markus Unterwaditzer wrote:
You're right, that is different.
|
In #331 I've decided to explicitly require the collections parameter for now. We may re-add a default later on, but I fear that simply changing the default would break user's configuration in too subtle ways. |
0.9 is released without default, next release will be with new default. |
Removing from 1.0 milestone since re-adding a default is not a breakage. However, I wonder if we ever should readd a default at all. |
null
implies that the user entered direct URLs and paths to a collection. This is useful behavior in the case where collection discovery is completely broken on the server, but it is only rarely used and therefore a bad default.We have two options:PLAN:["from a", "from b"]
1.0Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: