Skip to content

refactor: move WoT definitions up in a single place #502

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
wants to merge 1 commit into from

Conversation

danielpeintner
Copy link
Contributor

@danielpeintner danielpeintner commented Sep 29, 2023

Note: gets rid of partial namespace definitions since they don't seem to work

fixes validation issues mentioned in #477 and https://github.com/w3c/respec/issues/4447


Preview | Diff

Note: gets rid of partial namespace definitions since they don't seem to work
@zolkis
Copy link
Contributor

zolkis commented Sep 29, 2023

Doesn't change any Web IDL, only the prose.

Having conformance classes has been quite a consensus in the group. Not everyone must implement discovery, for instance.
With this change, the normative of the spec (currently a Note) applies to all functionality together at once.

I don't see the benefit of this and should be discussed / agreed in an issue before a PR.

@danielpeintner
Copy link
Contributor Author

danielpeintner commented Sep 29, 2023

I don't see the benefit of this and should be discussed / agreed in an issue before a PR.

We got an email from @ashimura that unfortunately our current spec is causing issues for the publication, see https://github.com/w3c/respec/issues/4447

I just try to find a solution. Anyhow, @ashimura is trying something else and report back.

@zolkis
Copy link
Contributor

zolkis commented Sep 29, 2023

Oh, so we have the partial interface causing a problem? Might be a ReSpec issue?
There's no problems with partial declarations with bikeshed, for instance.

@relu91
Copy link
Member

relu91 commented Oct 2, 2023

Call 02/10:

@danielpeintner
Copy link
Contributor Author

As decided in #506 we keep using conformance classes -> closing PR

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

Successfully merging this pull request may close these issues.

3 participants