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

Expose spec repository in nightly.repository #105

Merged
merged 3 commits into from
Jun 18, 2020
Merged

Conversation

tidoust
Copy link
Member

@tidoust tidoust commented Jun 17, 2020

See #19. The repository is either retrieved from Specref or computed from the nightly URL.

See w3c#19. The repository is either retrieved from Specref or computed from the
nightly URL.
@tidoust tidoust requested a review from dontcallmedom June 17, 2020 22:41
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.

I script-checked the following:

  • each repo listed in index.json appears in validate-repo (to make sure we haven't missed potential renames) - with the caveat that validate-repo doesn't cover whatwg, Khronos, webbluetoothcg and webidl
  • each drafts.csswg.org spec has an associated directory in w3c/csswg-drafts (which is only true if you also add the version number to the nightly url)

I detected one bug (trailing / in repo URL) and one likely needed improvement (case of the github org name)

index.json Outdated Show resolved Hide resolved
index.json Outdated Show resolved Hide resolved
@tidoust tidoust changed the title Expose spec repostiory in nightly.repository Expose spec repository in nightly.repository Jun 18, 2020
index.json Outdated
@@ -1056,7 +1056,7 @@
},
"nightly": {
"url": "https://wicg.github.io/frame-timing/",
"repository": "https://github.com/w3c/frame-timing"
"repository": "https://github.com/wicg/frame-timing"
Copy link
Member

Choose a reason for hiding this comment

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

ha, this uncovered another bug - I guess in my check of repo existence, I should have checked they were not archived either

@tidoust tidoust merged commit c1a2bc5 into w3c:master Jun 18, 2020
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.

2 participants