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.

Sign up for GitHub

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

Jump to bottom

Remove assertion/test annotation system #119

Closed
annevk opened this issue Feb 20, 2017 · 0 comments
Closed

Remove assertion/test annotation system #119

annevk opened this issue Feb 20, 2017 · 0 comments

Comments

@annevk
Copy link
Member

annevk commented Feb 20, 2017

@hallvors put a bunch of effort into annotation, linking tests and assertions together. Unfortunately it's not maintained and nobody else really picked it up.

There's an old PR here over two years old that demonstrates that a bit: web-platform-tests/wpt#1272.

I'm inclined to remove the whole system as it seems to have served its purpose in organizing the test suite.

I'll leave this issue open for a bit to give folks some time to comment though.
@annevk annevk mentioned this issue Feb 20, 2017
Closed
annevk added a commit that referenced this issue Feb 23, 2017
@annevk
Meta: remove annotation system and update deploy infrastructure
c3a1655 
Now with the reduplicated deploy.sh script and no more commit snapshots
for commits not on master. See whatwg/meta#11.

Fixes #119.
@annevk annevk mentioned this issue Feb 23, 2017
Merged
annevk added a commit that referenced this issue Feb 23, 2017
@annevk
Meta: remove annotation system and update deploy infrastructure
96e3512 
Now with the reduplicated deploy.sh script and no more commit snapshots
for commits not on master. See whatwg/meta#11.

Fixes #119.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@annevk