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

Some updates based on #1008 #1082

Merged
merged 1 commit into from
Oct 11, 2023
Merged

Some updates based on #1008 #1082

merged 1 commit into from
Oct 11, 2023

Conversation

tidoust
Copy link
Member

@tidoust tidoust commented Oct 10, 2023

  • Add WebXR Plane Detection
  • Add WebXR Meshing API
  • Add WebXR Mesh Detection Module
  • Add WebBluetooth Scanning API
  • Add Managed Configuration API
  • Add MathML4, as non browser spec
  • Monitor Private Aggregation API (pending w3c.json at least)
  • Ignore a few additional specs (not required as these specs are not found but that makes the rationale explicit)

- Add WebXR Plane Detection
- Add WebXR Meshing API
- Add WebXR Mesh Detection Module
- Add WebBluetooth Scanning API
- Add Managed Configuration API
- Add MathML4, as non browser spec
- Monitor Private Aggregation API (pending w3c.json at least)
- Ignore a few additional specs (not required as these specs are not found
but that makes the rationale explicit)
@@ -153,6 +153,14 @@
"https://immersive-web.github.io/anchors/",
"https://immersive-web.github.io/model-element/",
"https://immersive-web.github.io/raw-camera-access/",
{
"url": "https://immersive-web.github.io/real-world-geometry/plane-detection.html",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The IDL of that spec is incomplete (missing [Exposed] on the interface), which IIUC will block the IDL package release down the line. I'm curious if there is a better way to deal with this than just going through the cascade of releases.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not have a fully satisfactory solution to propose today, either:

  1. we put the spec in the monitor list for now since we know that the IDL is invalid; or
  2. we create a patch; or
  3. we wait until the IDL gets fixed

I'm surprised that we did not seem to have any open issue on the topic, even though we've been floating the idea to improve the curation process in that regard for some time. I created w3c/webref#1055 to track this.

Another option could be to automate checks a bit more when we add a spec to browser-specs, e.g., by running Reffy and tests on the spec and prevent addition of the spec while checks fail. That seems nice in theory but more complex in practice. Also, that wouldn't solve the problem for specs that start to fail after some time.

Copy link
Member

@dontcallmedom dontcallmedom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! let's proceed with this and we'll deal with the consequences as they emerge this time around; we now have w3c/webref#1055 to look into a more robust approach

@tidoust tidoust merged commit 366134e into main Oct 11, 2023
1 check passed
@tidoust tidoust deleted the add-specs branch October 11, 2023 09:52
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.

None yet

2 participants