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

Rename CamelCase repo names #112

Closed
6 of 7 tasks
foolip opened this issue May 27, 2020 · 10 comments
Closed
6 of 7 tasks

Rename CamelCase repo names #112

foolip opened this issue May 27, 2020 · 10 comments

Comments

@foolip
Copy link
Member

foolip commented May 27, 2020

#42 updated https://github.com/WICG/admin/wiki/Process-to-transfer-a-repo to say:

Admin: Check the repo name adheres to our repo naming policy: lowercase, dashes (foo-bar, not fooBar!). If it doesn't adhere, ask the owner user to rename their repo.

A bunch of repos still don't adhere to this convention. Here's a script to enumerate them:

const {Octokit} = require('@octokit/rest');
(async () => {
  const octokit = new Octokit();
  const options = octokit.repos.listForOrg.endpoint.merge({org: 'WICG'});
  const repos = await octokit.paginate(options);
  for (const repo of repos) {
    if (repo.archived) {
      continue;
    }
    if (!repo.name.match(/^[a-z0-9-.]+$/)) {
      console.log(repo.name);
    }
  }
})();

And the resulting list:

@foolip
Copy link
Member Author

foolip commented May 27, 2020

@marcoscaceres perhaps you can help with this?

The reason I keep noticing is that this ends up as a mismatch with directory names in WPT.

@tabatkins
Copy link

I'll need to do some renaming in Shepherd when this happens. Not an issue, just alerting myself to this thread.

@marcoscaceres
Copy link
Contributor

Sorry, missed this... will get on it.

@marcoscaceres
Copy link
Contributor

Ok, did the renames... will file bugs for each to let them know.

@tidoust
Copy link
Contributor

tidoust commented Jun 3, 2020

Why an uppercase I for Input-device-capabilities?

It would be good to update biblio.json for specs that were already in Specref, typically Input Device Capabilities. And a redirect at least for this one, as it has been referenced in multiple documents.

For the CSS Parser API, am I right to think that the work has moved to the Houdini Task Force? The README and the spec in the WICG repo could perhaps mention it. And Specref needs to be updated as it still points at the WICG URL. I'll take care of it.

@foolip
Copy link
Member Author

foolip commented Jun 3, 2020

I ran the script again and found the same as @tidoust, "Input-device-capabilities". I've unchecked that in the checklist.

foolip added a commit to w3c/browser-specs that referenced this issue Jun 3, 2020
The repo was renamed, see WICG/admin#112.
foolip added a commit to w3c/browser-specs that referenced this issue Jun 3, 2020
tidoust added a commit to tidoust/browser-specs that referenced this issue Jun 3, 2020
This update adjusts the URLs of:
- CSS Parser API
- Input Device Capabilities
- Scroll To Text Fragment

See discussion in WICG/admin#112.

Shortnames will change accordingly.
tidoust added a commit to tidoust/wicg.github.io that referenced this issue Jun 3, 2020
marcoscaceres pushed a commit to WICG/wicg.github.io that referenced this issue Jun 3, 2020
@tidoust
Copy link
Contributor

tidoust commented Jun 3, 2020

For the CSS Parser API, am I right to think that the work has moved to the Houdini Task Force? The README and the spec in the WICG repo could perhaps mention it. And Specref needs to be updated as it still points at the WICG URL. I'll take care of it.

Hmm, actually, the houdini draft points at the WICG as the place where incubation happens, so I suspect that the work has not really moved yet. In which case, it would make sense to re-insert an entry for it in biblio.json.

tidoust added a commit to tidoust/admin that referenced this issue Jun 3, 2020
See discussion in WICG#112. Work still seems to be done in the WICG (or the
development of the spec has stalled). In particular, actual spec content is in
the WICG spec, and not in the Houdini TF version.
@marcoscaceres
Copy link
Contributor

@tidoust reminded me to add the redirects over in the wicg.github.io repo.

tidoust pushed a commit to w3c/browser-specs that referenced this issue Jun 3, 2020
tidoust pushed a commit to w3c/browser-specs that referenced this issue Jun 3, 2020
@tidoust
Copy link
Contributor

tidoust commented Jun 3, 2020

GitHub seems lost with the lowercase rename for input-device-capabilities. The repo correctly appears at https://github.com/wicg/input-device-capabilities but AFAICT the spec is still published at https://wicg.github.io/Input-device-capabilities (with an uppercase I). It may be worth switching off and on again GitHub Pages for the repo...

@marcoscaceres
Copy link
Contributor

input-device-capabilities is fixed now.

marcoscaceres pushed a commit that referenced this issue Jun 3, 2020
See discussion in #112. Work still seems to be done in the WICG (or the
development of the spec has stalled). In particular, actual spec content is in
the WICG spec, and not in the Houdini TF version.
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

No branches or pull requests

4 participants