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

Update review of monitored specs #534

Merged
merged 9 commits into from
Mar 16, 2022
Merged

Update review of monitored specs #534

merged 9 commits into from
Mar 16, 2022

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Mar 1, 2022

The following specs have been updated since the last review:

Job accidently re-created the file because it ran under Node.js v14. That should
not happen again as jobs now all run using Node.js v16.
Support for the API has shipped in Chrome. No known support from other browser
vendors, but no objections either. Also see the intent to ship:
https://groups.google.com/a/chromium.org/g/blink-dev/c/2pjQ3O2GzDA/m/Uy-eRkB5BQAJ

Diff:

```json
{
  "added": [
    {
      "url": "https://wicg.github.io/digital-goods/",
      "seriesComposition": "full",
      "shortname": "digital-goods",
      "series": {
        "shortname": "digital-goods",
        "currentSpecification": "digital-goods",
        "title": "Digital Goods API",
        "shortTitle": "Digital Goods API",
        "nightlyUrl": "https://wicg.github.io/digital-goods/"
      },
      "organization": "W3C",
      "groups": [
        {
          "name": "Web Platform Incubator Community Group",
          "url": "https://www.w3.org/community/wicg/"
        }
      ],
      "nightly": {
        "url": "https://wicg.github.io/digital-goods/",
        "repository": "https://github.com/WICG/digital-goods",
        "sourcePath": "spec.bs",
        "filename": "index.html"
      },
      "title": "Digital Goods API",
      "source": "spec",
      "shortTitle": "Digital Goods API",
      "categories": [
        "browser"
      ]
    }
  ],
  "updated": [],
  "deleted": []
}
```
While not yet perfect, the Editor's Draft now uses ReSpec.

Diff:

```json
{
  "added": [
    {
      "url": "https://w3c.github.io/PNG-spec/",
      "seriesComposition": "full",
      "shortname": "PNG-spec",
      "series": {
        "shortname": "PNG-spec",
        "currentSpecification": "PNG-spec",
        "title": "Portable Network Graphics (PNG) Specification (Third Edition)",
        "shortTitle": "PNG",
        "nightlyUrl": "https://w3c.github.io/PNG-spec/"
      },
      "organization": "W3C",
      "groups": [
        {
          "name": "Portable Network Graphics (PNG) Working Group",
          "url": "https://www.w3.org/groups/wg/png/"
        }
      ],
      "nightly": {
        "url": "https://w3c.github.io/PNG-spec/",
        "repository": "https://github.com/w3c/PNG-spec",
        "sourcePath": "index.html",
        "filename": "index.html"
      },
      "title": "Portable Network Graphics (PNG) Specification (Third Edition)",
      "source": "spec",
      "shortTitle": "PNG",
      "categories": [
        "browser"
      ]
    }
  ],
  "updated": [],
  "deleted": []
}
```
Underlying specs have been added to the list already.
The spec has been re-published with the right status since last review.
Support for the API has shipped in Chrome, with positive signals from Firefox:
mozilla/standards-positions#565

Diff:

```json
{
  "added": [
    {
      "url": "https://wicg.github.io/capability-delegation/spec.html",
      "seriesComposition": "full",
      "shortname": "capability-delegation",
      "series": {
        "shortname": "capability-delegation",
        "currentSpecification": "capability-delegation",
        "title": "Capability Delegation",
        "shortTitle": "Capability Delegation",
        "nightlyUrl": "https://wicg.github.io/capability-delegation/spec.html"
      },
      "organization": "W3C",
      "groups": [
        {
          "name": "Web Platform Incubator Community Group",
          "url": "https://www.w3.org/community/wicg/"
        }
      ],
      "nightly": {
        "url": "https://wicg.github.io/capability-delegation/spec.html",
        "repository": "https://github.com/WICG/capability-delegation",
        "sourcePath": "spec.bs",
        "filename": "spec.html"
      },
      "title": "Capability Delegation",
      "source": "spec",
      "shortTitle": "Capability Delegation",
      "categories": [
        "browser"
      ]
    }
  ],
  "updated": [],
  "deleted": []
}
```
@tidoust tidoust marked this pull request as ready for review March 16, 2022 13:48
@tidoust
Copy link
Member

tidoust commented Mar 16, 2022

@dontcallmedom Not sure how you'd like to manage Capture Handle specs (https://wicg.github.io/capture-handle/). The Identity spec seems ready to be included. The Actions spec seems to still require updates. Should we wait until next review window?

@dontcallmedom
Copy link
Member

capture handle has now moved to w3c (having been adopted by the WebRTC WG) and is in our "new spec" queue; I've been waiting until the action spec gets its update to push my PR that add both specs to our list.

In the context of this PR, it can be left in the monitor list indeed, as my PR will remove it from there.

@tidoust
Copy link
Member

tidoust commented Mar 16, 2022

OK, then I think we're good to go with the updates I made to the PR.

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 a lot for going through the list! we probably need to find a way to make this a bit less overwhelming…

@tidoust tidoust merged commit 80d995a into main Mar 16, 2022
@tidoust tidoust deleted the monitor-update-1646093389 branch March 16, 2022 14: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.

3 participants