Skip to content
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

Addresses #151 ("Add TTV/TTXV test files") #152

Merged
merged 1 commit into from Feb 4, 2016

Conversation

@palemieux
Copy link
Contributor

commented Feb 3, 2016

Closes #151

@palemieux palemieux added the pr open label Feb 3, 2016
@palemieux palemieux self-assigned this Feb 3, 2016
@palemieux palemieux added this to the imsc1-cr4 milestone Feb 3, 2016
@nigelmegitt

This comment has been minimized.

Copy link
Contributor

commented Feb 3, 2016

No objection from me to including these contributions (for which thank you @skynavga) as long as they're not included in the test suite for which we need passing implementations to exit CR.

@palemieux

This comment has been minimized.

Copy link
Contributor Author

commented Feb 3, 2016

The proposal is to include them as reference test files, alongside the test files contributed by DECE

@skynavga

This comment has been minimized.

Copy link
Contributor

commented Feb 3, 2016

it may be a good idea to reorganize the test suite, perhaps as follows:

test/
w3c/
contrib/
dece/
skynav/

putting the "formal" testsuite files into test/w3c

On Wed, Feb 3, 2016 at 8:23 AM, Pierre-Anthony Lemieux <
notifications@github.com> wrote:

The proposal is to include them as reference test files, alongside the
test files contributed by DECE


Reply to this email directly or view it on GitHub
#152 (comment).

@palemieux palemieux referenced this pull request Feb 3, 2016
@palemieux

This comment has been minimized.

Copy link
Contributor Author

commented Feb 3, 2016

I like the proposed structure, and have created #153

palemieux added a commit that referenced this pull request Feb 4, 2016
Addresses #151 ("Add TTV/TTXV test files")
@palemieux palemieux merged commit 11b17a7 into master Feb 4, 2016
@palemieux palemieux deleted the issue-151 branch Feb 10, 2016
@palemieux palemieux modified the milestones: imsc1-cr4, imsc1-pr Feb 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.