Skip to content

Temporarily Remove TCK Implementation #39

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

Closed
benjchristensen opened this issue Apr 24, 2014 · 6 comments
Closed

Temporarily Remove TCK Implementation #39

benjchristensen opened this issue Apr 24, 2014 · 6 comments

Comments

@benjchristensen
Copy link
Contributor

I propose temporarily removing the TCK implementation and that we stop working on it until we arrive at a fairly stable agreement on the API and contract.

There is significant discussion going on in #19 and #37 that is changing the contract, and I expect further changes beyond these. The TCK impedes rapid iteration and confuses things until the contract is clearly defined.

Do we have agreement to delete what's currently there (and about to be obsolete) until we reach an agreement on contract?

@jbrisbin
Copy link

+1

@rkuhn
Copy link
Member

rkuhn commented Apr 24, 2014

+1

Before bringing it back we need to have at least two reference implementations of the new semantics so that we know that the TCK verifies the right thing.

rkuhn added a commit that referenced this issue Apr 24, 2014
Squash of #37
Work as a result of discussion in #19
Removes TCK implementation so it is not out of sync with API as per #39
@patriknw
Copy link
Contributor

+1

@benjchristensen
Copy link
Contributor Author

Based on agreement above I have submitted and will now merge a pull request to remove the TCK implementation but leave the folder structure in place.

@smaldini
Copy link
Contributor

Will try to be candidate to the reference implementation :)

@ktoso
Copy link
Contributor

ktoso commented Apr 25, 2014

removing it makes sense for now, +1 @benjchristensen :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants