-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
Formal specification for ckan clients #956
Comments
Oh my, I shuddered at the word "official". Perhaps we can call it extant? :)
I agree with this. It makes perfect sense to say something like "a client must not allow two or more conflicting mods to be installed" when we're talking about conflicts, for example. This is also nice because it means we can write a test suite that directly exercises the spec. It's worth noting that a lot of things the clients do will be out of the scope of the spec. Relationship resolution, for example, can be done in numerous ways, and it would be detrimental to enforce clients to work one way or the other. |
One more reason to formalize what a client must do (and what not) in order to be officially compatible with ckan :) Personally I would hesitate to say that a client "must not" allow conflicting mods (as this prohibits a --force option if a client wishes to implement that). Later today I will start on a first draft :) |
Sorry to be a stick in the mud, but what's the value to be provided here in practical terms? It mostly seems to me that this would create distractions rather than advancing the project...
|
Not likely to happen, even if it was a good idea. |
Currently, the spec only defines the structure of the metadata.
However, there is no formal specification for how the client must behave with the metadata: as a result, other implementations of the client might end up implementing a different behaviour than the official client.
I propose that we formalize the desired behaviour that the clients should implement: each review of the metadata spec should also come with a specification of the expected behaviour.
The text was updated successfully, but these errors were encountered: